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

[Enhancement II] Compatible processed types #17

Merged
merged 43 commits into from
Jul 19, 2024

Conversation

CodyCBakerPhD
Copy link
Member

Plane segmenations are not compatible with the new ImagingSpace here

RoiResponseSeries should be fine though since it just has a dynamic table region that can point to anything

@CodyCBakerPhD CodyCBakerPhD self-assigned this Jun 27, 2024
Specify 'whole-brain' if the entire brain is strictly contained within the space.
Specify 'whole brain' if the entire brain is strictly contained within the space.
Copy link
Member Author

Choose a reason for hiding this comment

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

Also sneaking this change in

Copy link
Member Author

Choose a reason for hiding this comment

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

From @alessandratrapani, has not seen the version with the dash, but intuitively wanted 'Whole Brain'

@CodyCBakerPhD CodyCBakerPhD changed the title [Enhacement III] Compatible processed types [Enhacement II] Compatible processed types Jul 12, 2024
@CodyCBakerPhD CodyCBakerPhD changed the title [Enhacement II] Compatible processed types [Enhancement II] Compatible processed types Jul 12, 2024
Base automatically changed from add_multichannel_volume to main July 15, 2024 19:27
@CodyCBakerPhD CodyCBakerPhD marked this pull request as ready for review July 18, 2024 17:24
@CodyCBakerPhD
Copy link
Member Author

@alessandratrapani I added tests as well as the other items we discussed

Copy link
Collaborator

@alessandratrapani alessandratrapani left a comment

Choose a reason for hiding this comment

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

LGTM!

@CodyCBakerPhD CodyCBakerPhD merged commit fb8c627 into main Jul 19, 2024
33 checks passed
@CodyCBakerPhD CodyCBakerPhD deleted the all_previous_prs_and_variable_depth_static branch July 19, 2024 14:00
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.

2 participants