Skip to content
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

Upgrade BrainGlobe Atlas API package name #650

Merged
merged 4 commits into from
May 24, 2024
Merged

Conversation

yoda-vid
Copy link
Collaborator

@yoda-vid yoda-vid commented May 24, 2024

The brainglobe atlas API was renamed from bg-atlasapi to brainglobe-atlasapi (see brainglobe/brainglobe-atlasapi#176), including both the package name the corresponding top-level namespace. Setup now installs this new package, which updates versions from v1 to v2.

To add a couple registration but otherwise unrelated changes, this PR includes a couple other changes:

  • Registered images are now detected for transformed images, such as those in my_image_scale0pt1 format with registered images output to my_image. This avoid the need to use the --prefix flag when loading the image, though the flag is still needed when registering the image.
  • Registration parameters are now shown in the console when using ITK-Elastix, matching the behavior when using SimpleITK

yoda-vid added 3 commits May 24, 2024 14:37
Print out user-defined parameters, similar to output in SimpleITK mode.
Image filenames generated from transformations such as rescaling have required using the `--prefix` CLI argument to load registered images. Since these filenames have suffixes from the transformation, the filenames can be deconstructed to set this prefix value.
- Support the package name change while continuing to support the old package
- Install this package by default
@yoda-vid yoda-vid added enhancement New feature or request dependencies Issues/requests related to a dependency package labels May 24, 2024
@yoda-vid yoda-vid added this to the v1.6.0 milestone May 24, 2024
Fall back to older `bg-atlasapi` package.
@yoda-vid yoda-vid merged commit ee20927 into master May 24, 2024
36 checks passed
@yoda-vid yoda-vid deleted the upgrade_brainglobe branch May 24, 2024 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Issues/requests related to a dependency package enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant