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

Intialize Provenance Tracking #423

Draft
wants to merge 24 commits into
base: dev
Choose a base branch
from

Conversation

omar-rifai
Copy link
Contributor

@omar-rifai omar-rifai commented Sep 16, 2021

WIP towards a traceability feature for Clinica:

  • Define data model for PROV concepts
  • Serialize and de-serialize json-ld documents
  • Compose prov modules with pipelines
  • Log all attributes defined in BIDS prov
  • Homogenize inputs and outputs of all pipelines for use in prov

@pep8speaks
Copy link

pep8speaks commented Sep 16, 2021

Hello @omar-rifai! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 26:10: E111 indentation is not a multiple of four
Line 26:10: E117 over-indented
Line 33:55: E231 missing whitespace after ','
Line 33:98: W291 trailing whitespace

Comment last updated at 2022-03-14 16:07:38 UTC

@omar-rifai omar-rifai force-pushed the feat_init_traceability branch 2 times, most recently from c34018d to 3d25540 Compare September 23, 2021 07:51
@ghisvail
Copy link
Collaborator

ghisvail commented Jan 5, 2022

You gonna have to rebase onto the latest dev to pass CI

@omar-rifai omar-rifai changed the title Feat init traceability Intialize Provenance Tracking Jul 8, 2022
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.

3 participants