pnlbwh / dMRIharmonization

Multi-site dMRI harmonization
Other
43 stars 14 forks source link

TODO: structure the output statistics file #47

Closed tashrifbillah closed 4 years ago

tashrifbillah commented 4 years ago

multi-shell-dMRIharmonization:

Plots should be different, one for each bvalue:

meanFAstat_b{bshell_b}_ebarplot.png
meanFAstat_b{bshell_b}_boxplot.png

However,

Reference.csv (easy to append columns)
Target_before.csv (easy to append columns)
Target_after.csv (easy to append columns)
meanFAstat.csv (we should drop std meanFA, that way it would be easy to append columns)

Should be same for all bvalues.

tashrifbillah commented 4 years ago

Hint: Read existing DataFrame, append new columns, and write back.

tashrifbillah commented 4 years ago

Summary is saved as:

template/meanFAstat.csv:

image

Individual site statistics are saved as:

template/{siteName}*.csv

image

@suheyla2 , @yrathi , how do we like it now?

tashrifbillah commented 4 years ago

Two plots are saved:

  1. Conventional box plot with Q1, Q2, and Q3: image

  2. Error bar plot demonstrating mean and range of SD: image

Previously, I was saving only the former overriding the median (Q2) with mean. @sbouix remarked this can be misleading. So, the design has been revised and two plots are saved respecting the definition of each of the plots.

yrathi commented 4 years ago

Lets discuss this a bit more. I will stop by to chat with you about this.

Thanks Yogesh

On Feb 17, 2020, at 4:44 PM, Tashrif Billah notifications@github.com<mailto:notifications@github.com> wrote:

    External Email - Use Caution

Two plots are saved:

  1. Conventional box plot with Q1, Q2, and Q3: [image]https://user-images.githubusercontent.com/35086881/74688102-69d0cf00-51a4-11ea-84fe-56bc9f605666.png

  2. Error bar plot demonstrating mean and range of SD: [image]https://user-images.githubusercontent.com/35086881/74688132-853bda00-51a4-11ea-8ea7-e8e50de50bd3.png

Previously, I was saving only the former overriding the median (Q2) with mean. @sbouixhttps://github.com/sbouix remarked this can be misleading. So, the design has been revised and two plots are saved respecting the definition of each of the plots.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/pnlbwh/dMRIharmonization/issues/47?email_source=notifications&email_token=ABNK3IJRTHSCZPDEDKYOE53RDMAMRA5CNFSM4KJZKT3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL7XWGA#issuecomment-587168536, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABNK3IPGLMYOV3CO6J2PAT3RDMAMRANCNFSM4KJZKT3A.

The information in this e-mail is intended only for the person to whom it is addressed. If you believe this e-mail was sent to you in error and the e-mail contains patient information, please contact the Partners Compliance HelpLine at http://www.partners.org/complianceline . If the e-mail was sent to you in error but does not contain patient information, please contact the sender and properly dispose of the e-mail.

tashrifbillah commented 4 years ago

_boxplot is omitted. Only _ebarplot is generated.