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

ENH: Add bundle feature population math script #138

Merged

Conversation

jhlegarreta
Copy link
Contributor

@jhlegarreta jhlegarreta commented Jul 24, 2023

Add bundle feature population math script: allows to compute the

  • difference
  • stats
  • sum

for each bundle feature across participants according to the values
contained in the provided CSV files.

Add the corresponding test.

@ljod
Copy link
Member

ljod commented Oct 12, 2023

please modify script name so we can guess what it is used for. it sounds like it is for statistical analysis for a paper to me

@jhlegarreta jhlegarreta force-pushed the AddBundleFeaturePopulationMathScript branch 2 times, most recently from 3a40b14 to 0598537 Compare October 12, 2023 19:57
@jhlegarreta
Copy link
Contributor Author

jhlegarreta commented Oct 12, 2023

please modify script name so we can guess what it is used for.

Made clear the purpose in the script help.

it sounds like it is for statistical analysis for a paper to me

Could be, but I used to clearly state whether the results that we got from WMA for the SZBD project were the same or not doing a diff.

@jhlegarreta jhlegarreta force-pushed the AddBundleFeaturePopulationMathScript branch from 0598537 to a25a610 Compare October 12, 2023 19:58
Add bundle feature population math script: allows to compute the
- difference
- stats
- sum

for each bundle feature across participants according to the values
contained in the provided CSV files.

Add the corresponding test.
@jhlegarreta jhlegarreta force-pushed the AddBundleFeaturePopulationMathScript branch from a25a610 to 5974295 Compare October 12, 2023 19:58
@jhlegarreta jhlegarreta marked this pull request as ready for review October 12, 2023 20:00
@ljod ljod merged commit b3908da into SlicerDMRI:master Oct 19, 2023
0 of 4 checks passed
@jhlegarreta jhlegarreta deleted the AddBundleFeaturePopulationMathScript branch October 19, 2023 13:59
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

Successfully merging this pull request may close these issues.

2 participants