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

[E3B6] pipeline reproduction #160

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

Conversation

youennmerel
Copy link

@youennmerel youennmerel commented Feb 12, 2024

This Pull Request is related to issue #158

Changes proposed in this Pull Request:

  • WIP

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 bclenet changed the title [#158] E3B6 pipeline reproduction [E3B6] pipeline reproduction Feb 14, 2024
@bclenet
Copy link
Collaborator

bclenet commented Feb 14, 2024

Hello @youennmerel
Could you please add a comment to explain what you modified from the template, and what is left to do ?
Thx :)

@youennmerel
Copy link
Author

youennmerel commented Feb 14, 2024

Hello @youennmerel Could you please add a comment to explain what you modified from the template, and what is left to do ? Thx :)

Hi @bclenet,
I've done it on the linked issue.

  • Template template_spm.py has been used to initiate team_E3B6.py
  • For now, only the following functions can be considered functional
    • __init__
    • get_preprocessing
    • get_run_level_analysis (not used with SPM)

@bclenet
Copy link
Collaborator

bclenet commented Feb 14, 2024

Cool ! Sorry for the useless message ;)

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