New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding JRC2018F_VNC templatebrain #42
Conversation
Hi @istvantaisz, Thanks for this. Can you
|
I also like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- remove the DS_store (I suggest adding it to your global gitignore – google will tell you how)
- Change name to
JRCVNC2018F
- rename
data-raw/JRC2018_VNC_FEMALE_4iso.nrrd
todata-raw/JRC2018_VNC_FEMALE_4iso.nhdr
- document the
JRCVNC2018F
andJRCVNC2018F.surf
objects. See https://github.com/natverse/nat.flybrains/blob/master/R/JRC2018.R for an example. You can put the documentation forJRCVNC2018F
in the same file.
Hi @jefferis I've made the above changes. The surface wasn't generated exactly as the others described in the documentation, I will update the .surf file based on that |
Hi @jefferis I've made the above changes.
Great!
The surface wasn't generated exactly as the others described in the documentation, I will update the .surf file based on that
How was it different? I normally try to keep them fairly small 18-25k vertices for convenient plotting. I also downsample/smooth the original image and then smooth the surface. There should be some hx Amira script files which give the gist if you want to redo anything.
|
I updated the surface following the method used for the brains and changed the documentation accordingly. |
Thanks a lot @istvantaisz. A good test is whether you can use
The new mesh now passes that test, so looks good to me. |
hm, true. I suspect they are coming form the large axon tracts that were very low intensity on the image. |
* this should probably be fixed in nat
I think we can leave those blobs, they are presumably non-neuropil areas. There could be interest in removing them for some purposes (and perhaps closing some of the in pouches). But no need now. Thanks again for the PR. |
No description provided.