Skip to content

Read FC neurons#44

Merged
jefferis merged 22 commits intomasterfrom
readneurons
Aug 29, 2019
Merged

Read FC neurons#44
jefferis merged 22 commits intomasterfrom
readneurons

Conversation

@alexanderbates
Copy link
Copy Markdown
Collaborator

  • read flycircuit neurons and get flycircuit neuron IDs
  • updates to README
  • Not sure how to solve error thrown by Rmd Check

@jefferis
Copy link
Copy Markdown
Collaborator

(just a note that I still have work to do on this branch ...)

alexanderbates and others added 17 commits August 29, 2019 01:00
GJ mods:

* move registrations to a folder included in package
* inst/extdata (perhaps against convention) is ignored during package
  build because it is in .Rbuildignore. This was likely the reason for
  the issue with the location of registration files earlier
* remove duplicate registration file
* had both gzipped and plain text version
* was missing some ids and making extra page requests
* dontrun example – it's very long if pkgdown tries to run it
* and details of v1.0 v 1.2 ids
* don't run long running example
* this is the pattern in the rest of the package
s
@jefferis jefferis merged commit 9b699dc into master Aug 29, 2019
@jefferis
Copy link
Copy Markdown
Collaborator

NB I rebased to avoid the addition of a duplicate copy of one of the warping registrations.

@jefferis
Copy link
Copy Markdown
Collaborator

Also note additional fixes in 7b59db6 for fc_page

@jefferis
Copy link
Copy Markdown
Collaborator

and that flycircuit_* was renamed to fc_* to match the rest of the package. @alexanderbates if this rename is a problem, you can potentially handle this by making a deprecated function in catnat that calls the correct flycircuit function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants