neurobagel / digest

Web dashboard for neuroimaging-derived and phenotypic datasets
https://digest.neurobagel.org
MIT License
2 stars 2 forks source link

[BUG] extra requirements for same subjects and sessions across pipelines seems too restrictive #61

Closed Remi-Gau closed 7 months ago

Remi-Gau commented 1 year ago

Is there an existing issue for this?

Expected Behavior

I would expect the bagel.csv I attached to work with the dashboard.

bagel.zip

Current Behavior

No response

Error message

Getting the following error.

Error: The pipelines in bagel.csv do not have the same number of subjects and sessions. Please try again.

Environment

No response

How to reproduce

try to use the bagel.csv attached

Anything else?

bagel.zip

Note that commenting the following lines to remove the extra check, did let the dashboard use the csv and did not throw any other error I could detect.

https://github.com/neurobagel/proc_dash/blob/03ab3021252504a3ec78617166e7ea618970bab2/proc_dash/utility.py#L158-L159

github-actions[bot] commented 1 year ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. We have applied the stale-issue label to indicate that this issue should be reviewed again and then either prioritized or closed.

github-actions[bot] commented 1 year ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. We have applied the stale-issue label to indicate that this issue should be reviewed again and then either prioritized or closed.

nikhil153 commented 8 months ago

@alyssadai - reviving this issue since it has become rather important for QPN dataset.

Currently seeing this error when phenotypic digest-bagel is uploaded containing unequal number of sessions per assessments (i.e. pipelines).

It's not possible for QPN to meet this criterion because of ad-hoc nature of visits.

Error: The pipelines in the selected CSV do not have the same number of subjects and sessions. Please try again.