-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Enhancement II] Compatible processed types #17
Conversation
[Enhacement II] Testing
…able_depth_static
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. |
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.
Also sneaking this change in
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.
From @alessandratrapani, has not seen the version with the dash, but intuitively wanted 'Whole Brain'
…able_depth_static
for more information, see https://pre-commit.ci
…github.com/catalystneuro/ndx-microscopy into all_previous_prs_and_variable_depth_static
for more information, see https://pre-commit.ci
…github.com/catalystneuro/ndx-microscopy into all_previous_prs_and_variable_depth_static
for more information, see https://pre-commit.ci
@alessandratrapani I added tests as well as the other items we discussed |
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.
LGTM!
Plane segmenations are not compatible with the new
ImagingSpace
hereRoiResponseSeries
should be fine though since it just has a dynamic table region that can point to anything