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

Don't require 'roi' column, use roi-sets instead #7

Open
stuarteberg opened this issue Oct 5, 2023 · 0 comments
Open

Don't require 'roi' column, use roi-sets instead #7

stuarteberg opened this issue Oct 5, 2023 · 0 comments

Comments

@stuarteberg
Copy link
Member

The first version of this tool exported a flat connectome and connectivity reports. Those assumed that the input data contained just one column for ROIs, named roi. Since then, the code has been altered to support multiple ROI columns, each referred to as an "roi set". The ROI sets need not be provided in the input synpase table; instead, they can be determined via ROI masks or segmentations in DVID.

But the original flat-connectome and reports functionality both still assume exactly one ROI column, named roi, and they don't use the user-defined roi-sets at all. We should change that somehow. Maybe the easiest thing would be to allow the config to name ONE of the roi-sets to use for the reports and.or connectome export.

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

1 participant