-
Notifications
You must be signed in to change notification settings - Fork 30
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
Fixing admba #322
Fixing admba #322
Conversation
Result of mob-programming! Co-authored-by: Viktor Plattner <[email protected]> Co-authored-by: sfmig <[email protected]>
0d3fc1f
to
03c4a13
Compare
Hey @viktorpm I haven't changed all the resolutions in the config yet, but we have concluded that
|
@@ -314,8 +319,13 @@ def create_atlas( | |||
species="Mus musculus", | |||
atlas_link="https://search.kg.ebrains.eu/instances/8ab25629-bdac-47d0-bc86-6f3aa3885f29", | |||
atlas_file_url="https://data.kg.ebrains.eu/zip?container=https://object.cscs.ch/v1/AUTH_4791e0a3b3de43e2840fe46d9dc2b334/ext-d000023_3Drecon-ADMBA-E11pt5_pub", | |||
orientation="lsa", | |||
resolution=(16, 16, 20), | |||
orientation="rsa", |
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 is the correct way around, others need to be adapted!
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.
Amazing - thank you @viktorpm
What is this PR
What does this PR do?
Fixes admba meshes and annotations mismatch, implements pooch to handle atlas downloading
How has this PR been tested?
Tested locally, and atlases were regenerated on the HPC
Is this a breaking change?
No
Does this PR require an update to the documentation?
No
Checklist: