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

Add reproducibility info to pipeline status page #123

Merged
merged 30 commits into from
Nov 14, 2023

Conversation

bclenet
Copy link
Collaborator

@bclenet bclenet commented Oct 10, 2023

Changes proposed in this Pull Request:

  • Adding the following columns to the work progress status page of the wiki :
    • excluded from NARPS analysis (yes or no)
    • reproducibility : rating (out of 4 ⭐) on the reproducibility of the pipeline + textual information about difficulties that may occur while reproducing the pipeline

Checklist:

  • Descriptive title
  • Targets the main branch
  • Changes are functional
  • My code is explicit and comments were added to it
  • Code conforms with PEP8
  • Tests were added for the changes and they complete successfully
  • Existing tests were updated (if needed) and they complete successfully
  • Documentation was updated

@bclenet
Copy link
Collaborator Author

bclenet commented Oct 17, 2023

Hi @elodiegermani !

As in this PR, I try to create alternatives to the shared google spreadsheet analysis_pipelines_for_analysis, so that it's easier to parse and display information about :

  • team's pipeline descriptions
  • comments about the pipeline from the NARPS analysis
  • reproducibility of the pipelines

For now the latter part is incomplete because I'm not sure why some pipelines are marked as "not reproducible" in the original spreadsheet... Would you be able to tell me more about that ?

Many thanks :)

@bclenet bclenet marked this pull request as ready for review November 14, 2023 14:25
@bclenet bclenet merged commit 7b2e33a into Inria-Empenn:main Nov 14, 2023
5 checks passed
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.

1 participant