-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update stats plots, add longitudinal sample size calculation #98
Open
PaulBautin
wants to merge
42
commits into
master
Choose a base branch
from
graph
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+309
−179
Open
Changes from 1 commit
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
fdd81a1
update plots to match manuscript
PaulBautin a1a3d3c
add:
PaulBautin d48a7b8
- update README for longitudinal study sample size
PaulBautin 645dcc1
update ref in csa_rescale_stat for sample size computation
PaulBautin f4f5c4b
correct rescale_estimated_subject in README
PaulBautin 79e1269
Create output folder if does not exist
jcohenadad 08b1d01
change std to var in sample size
PaulBautin c0f0596
Added TODOs
jcohenadad deb15c6
Merge remote-tracking branch 'origin/graph' into graph
PaulBautin de28fc3
- correct sample size formula
PaulBautin f72e107
- correct diff formula by integrating transformation variability (bef…
PaulBautin d39cc25
- correct sample size formula
PaulBautin ab54398
- remove replacement in diff formula for df_sub
PaulBautin a67f773
- use absolute for mean diff and make difference subject dependant
PaulBautin ab1eae1
- remove diff abs because useless (SD uses squared difference anyway)
PaulBautin 7874439
- remove diff abs because useless
PaulBautin e599ab7
- change formula for sample size
PaulBautin 05d7fcd
- Monte Carlo simulation for between group sample size computation
PaulBautin f50f18a
- add comments for sample size function
PaulBautin be0ebb4
- improve comments for formula var and var_diff
PaulBautin 31d2ab1
- limit usage of rescale_area to plots
PaulBautin 02c47e7
- remove rounding before boxplots for csa and atrophy
PaulBautin bb990c8
- scatter colorbar takes discrete values
PaulBautin 7676fcb
- compute sample size using 500 itterations
PaulBautin 08ff3a5
- update sample size plot to match article
PaulBautin d35b6f2
- remove ceil sur chaque calcul de sample size
PaulBautin 120fa41
add egg info
sandrinebedard 773c191
change sct_deepseg for sct_deepseg_sc
sandrinebedard e745a3c
change disc file label
sandrinebedard 25524fd
add fix for qform sfrom and fix typo disc labels
sandrinebedard 656e174
change for sct
sandrinebedard c849a24
modify for python 3.9 compatibility
sandrinebedard 9511c21
setup for compute canada
sandrinebedard 96c1b1d
rsync disc file
sandrinebedard 08bfc54
rm qc from sct_deepseg
sandrinebedard 6e36cbf
fix missing eextension
sandrinebedard a5942d4
remove extra QC
sandrinebedard 28b8c78
remove one qc report
sandrinebedard 5d37e0a
remove all QC reports
sandrinebedard bc3e882
remove all qc reports
sandrinebedard 0b40fb6
add logging
sandrinebedard d7c5bf9
Merge branch 'sb/update-for-ca-python3.9' into graph
sandrinebedard File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
- change formula for sample size
- add comments
commit e599ab7cfa1c848ab7469d58c6c394a3005e04e2
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add comment/explanation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added explanations for the sample size function in commit f50f18a