neurobagel / bulk_annotations

Retroactively annotate a large number of BIDS datasets at once
MIT License
0 stars 2 forks source link

Do explicit check for missing values for tool / continuous column types #28

Open surchs opened 1 year ago

surchs commented 1 year ago

we need to add an explicit check for missing values for the tool / continuous column types - similar to what we have done for discrete values.

According to our current data dictionary docs + reference example .json, any phenotypic column that's not an identifier column can have annotated missing values (I believe the CLI allows this as well). Based on the bulk_annotation_levels gsheet, I see that there's at least one assessment tool (e.g., ds003823 "ses-pre_task-SART" column) with a missing value annotated. Not sure how/if these are currently handled.

Steps:

_Originally posted by @alyssadai in https://github.com/neurobagel/bulk_annotations/pull/24#discussion_r1255037484_

github-actions[bot] commented 10 months ago

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days. We have applied the _flag:stale label to indicate that this issue should be reviewed again. When you review, please reread the spec and then apply one of these three options: