-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update original kim dev atlas details #280
Conversation
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.
This atlas is now superseded by its 1.0.0 version
Is this true. Firstly can v001 be superseded by v1. also they are in different spaces one in allen and one in MRI space, the allen space one is very useful for alternative analysis of allen registered data. Perhaps I wouldnt say its use is for reproducibility and archive reasons.
You're right, but we can't fix everything immediately. This at least makes the docs consistent. We're available for help if anyone gets confused. |
Perhaps I should reach out to the Kim group and ask them to clarify the versioning? |
Pretty sure 001 was the original release, when they had only created the p56 templates. 1.0.0 is the "official" full, first release of all timepoints. |
oh yeah for sure but I could contact them to ask if they meant "0.0.1" or how they would prefer we handle this |
They've previously referred to it as "v1". You're welcome to reach out, but I'm happy to just merge this and move on. |
@PolarBean I'm going to merge this as the 001/0.0.1 question was "decided" in #284, and this at least makes everything consistent. |
Follow up to brainglobe/brainglobe-atlasapi#493