Skip to content

Conversation

@saarah815
Copy link
Contributor

Description

What is this PR

  • Bug fix
  • Addition of a new feature (blog post)
  • Other

References

Closes: #395

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

Copy link
Member

@adamltyson adamltyson left a comment

Choose a reason for hiding this comment

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

Very nice, just a few small suggestions.

Thanks @saarah815!

Copy link
Member

Choose a reason for hiding this comment

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

I think a 2D section through the atlas might be better here. I don't think the 3D image of the template is that informative.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, thanks! Changed it to a 2D section of the annotations. Can do a side-by-side with the ref if that's better though.

Copy link
Member

Choose a reason for hiding this comment

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

I think side by side is nicer. Basically the same as you had before, but 2D.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cool - done :)

Copy link
Member

@adamltyson adamltyson left a comment

Choose a reason for hiding this comment

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

Lovely thanks. I will update the date to today, then this is good to go.

@adamltyson adamltyson merged commit d8bb595 into brainglobe:main Nov 6, 2025
2 checks passed
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.

[Feature] Add a blog about Dorr MRI atlas

2 participants