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

DOC: add a tutorial based on a group report I've developed #121

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

celprov
Copy link
Collaborator

@celprov celprov commented May 17, 2024

enh: add data necessary to run the tutorial

…eate an command line interface to generate a group report using nireports
enh: add data necessary to run the example
…in a previous PR. The tutorial simplifies the latter code by generating only two reportlets without dependence on MRIQC-generated IQMs.

fix: remove session not saved as examples from fMRI_duration_after_censoring.csv
@celprov celprov requested a review from mnoergaard May 17, 2024 14:05
Copy link

codecov bot commented May 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.20%. Comparing base (e0a30e6) to head (ef788d2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #121   +/-   ##
=======================================
  Coverage   59.20%   59.20%           
=======================================
  Files          20       20           
  Lines        2238     2238           
  Branches      398      398           
=======================================
  Hits         1325     1325           
  Misses        814      814           
  Partials       99       99           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@celprov
Copy link
Collaborator Author

celprov commented May 17, 2024

@mnoergaard It is not fully polished yet, but I was thinking we can start polishing it together.

celprov and others added 3 commits May 17, 2024 21:04
… session but one distribution per FC matrix row. This has been corrected.

enh: add a mention in the README pointing to the tutorial
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