search

CBICA / niCHART

The neuro-imaging brain aging chart [niCHART] is a comprehensive solution to analyze standard structural and functional brain MRI data across studies. [niCHART] and the associated pre-processing tools implement computational morphometry, functional signal analysis, quality control, statistical harmonization, data standardization, interactive visual
https://cbica.github.io/niCHART/
Other
13 stars 8 forks source link

App crashes if data set has only M or only F #185

Closed AbdulkadirA closed 2 years ago

AbdulkadirA commented 2 years ago

This can be reproduced by opening a data set that has only male or female participants (i.e. Sex=="M" or Sex=="F" for all participants).

ashishsingh18 commented 2 years ago

We have a IsValidData function https://github.com/CBICA/NiBAx/blob/101194e29cd3a4726b86656a0c5f5a8138d57b42/NiBAx/core/model/datamodel.py#L151 in our datamodel. Perhaps we should extend that to handle this as well.

AbdulkadirA commented 2 years ago

Having only M or only F in a data set is still valid. I think it’s the data set statistics plugin that crashes because there are no M or no F.

From: Ashish Singh @.> Sent: Friday, March 11, 2022 4:52 PM To: CBICA/NiBAx @.> Cc: Abdulkadir, Ahmed @.>; Author @.> Subject: [External] Re: [CBICA/NiBAx] App crashes if data set has only M or only F (Issue #185)

We have a IsValidData function https://github.com/CBICA/NiBAx/blob/101194e29cd3a4726b86656a0c5f5a8138d57b42/NiBAx/core/model/datamodel.py#L151 in our datamodel. Perhaps we should extend that to handle this as well.

— Reply to this email directly, view it on GitHubhttps://github.com/CBICA/NiBAx/issues/185#issuecomment-1065239582, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOELFTNNVSAIUFKNJU42K2DU7NTZTANCNFSM5QPWIO6Q. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.***>