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

may we please have all the anchorings filled by the propagation #30

Closed
PolarBean opened this issue Sep 20, 2023 · 3 comments
Closed

may we please have all the anchorings filled by the propagation #30

PolarBean opened this issue Sep 20, 2023 · 3 comments
Assignees

Comments

@PolarBean
Copy link
Member

When there are empty anchorings it leads to a lot of difficulty downstream in pynutil since we may incomplete anchorings, and incomplete segmentations. It has the opportunity to silently fail not quantifying the full brain as people might expect.

@PolarBean
Copy link
Member Author

Furthermore, since we no longer require exporting overlays for webnutil (which did output all the sections) this would make the behaviour consistent.

@Tevemadar
Copy link
Member

Some of those difficulties may go away if pynutil considers including the propagation code from https://github.com/Tevemadar/QuickNII-extras/tree/master/Python
Fresh from the oven, with a brand new bug fix, and the capability to take section numbers from the _s123 part of filenames (optionally existing nr field takes precedence).

@PolarBean
Copy link
Member Author

oh thats very useful, also for DeepSlice. Thank you

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

No branches or pull requests

2 participants