Skip to content

JRC2018Unisex Fly Brain Atlas - #808

Open
bahramani wants to merge 12 commits into
mainfrom
jrc2018u-neuropils-fly
Open

JRC2018Unisex Fly Brain Atlas#808
bahramani wants to merge 12 commits into
mainfrom
jrc2018u-neuropils-fly

Conversation

@bahramani

@bahramani bahramani commented May 14, 2026

Copy link
Copy Markdown
Member

Description

What is this PR?

  • Bug fix
  • Addition of a new feature
  • Other

Why is this PR needed?

This PR adds script for packaging the VFB ROI annotation on the JRC2018Unisex fly adult brain template into BrainGlobe format.

What does this PR do?

This PR adds atlas_scripts/jrc2018u_neuropils_fly.py.

References

Relates to #471 .

Main atlas/template source:

Main citation:

Bogovic JA, Otsuna H, Heinrich L, Ito M, Jeter J, et al. (2020) An unbiased template of the Drosophila brain and ventral nerve cord. PLOS ONE 15(12): e0236495. https://doi.org/10.1371/journal.pone.0236495

How has this PR been tested?

Is this a breaking change?

No. This PR only adds a new atlas generation script and should not change existing atlas behavior.

Does this PR require an update to the documentation?

No documentation update has been added in this PR.

Checklist

  • The code has been tested locally
  • Tests have been added to cover all new functionality (unit & integration)
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

Link to the .tar file

https://drive.google.com/file/d/1T0DniDHTzqChrmHZJbStn-zz2R03QRyu/view?usp=sharing

Some images of the atlas in napari

image image

@PolarBean
PolarBean marked this pull request as draft May 14, 2026 17:52
@PolarBean

Copy link
Copy Markdown
Member

Hey Amirreza, I just ran this and it seems just to be downloading the atlas files not generating them so ive converted it to draft for the moment. Let me know when its ready for review :)

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
2 Security Hotspots

See analysis details on SonarQube Cloud

@bahramani
bahramani marked this pull request as ready for review May 31, 2026 11:56
@bahramani bahramani changed the title JRC2018Unisex ROI Fly Atlas JRC2018Unisex Fly Brain Atlas Jul 12, 2026
Comment thread atlas_scripts/vfb_jrc2018u_neuropils_fly.py Outdated
return json.loads(term_info)


def _load_nrrd_array(nrrd_path):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same comment as on #817

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey, I adressed all of the comments.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey I think this comment hasnt been addressed

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes I missed that. Addressed!

Comment thread atlas_scripts/vfb_jrc2018u_neuropils_fly.py Fixed
@PolarBean

Copy link
Copy Markdown
Member

0.5189161um how is this derived? Given that 0.0000001um is the width of a virus I am sceptical of this level of precision

@PolarBean

Copy link
Copy Markdown
Member

lets keep it rounded to 3 decimal places

@bahramani

Copy link
Copy Markdown
Member Author

lets keep it rounded to 3 decimal places

The original 0.5189161 µm value came directly from the VFB NRRD header, but you are right that this level of precision is unnecessary and basically meaningless here. I have rounded the packaged resolution to 0.519 µm and updated the atlas name accordingly, while keeping the exact source spacing internally for coordinate conversion.

Comment thread atlas_scripts/vfb_jrc2018u_neuropils_fly.py Outdated
@sonarqubecloud

Copy link
Copy Markdown

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.

3 participants