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.

Sign up for GitHub

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

Jump to bottom

Use Nipreps-style configuration object throughout package #956

Closed
wants to merge 71 commits into from
Closed

Use Nipreps-style configuration object throughout package #956

wants to merge 71 commits into from

Conversation

tsalo
Copy link
Member

@tsalo tsalo commented Oct 3, 2023

Closes #816.

Changes proposed in this pull request

  • Replace workflow parameters with a config object, when feasible.

Documentation that should be reviewed

@tsalo tsalo added refactor Changes to the codebase that don't impact workflow inputs or outputs. maintenance Issues/PRs related to maintenance. Dependency versions, dependencies, etc. labels Oct 3, 2023
@tsalo
Copy link
Member Author

tsalo commented Oct 18, 2023

Now I'm getting all sorts of errors. Not sure what's going on.

@tsalo tsalo self-assigned this Oct 30, 2023
@tsalo tsalo closed this Feb 2, 2024
@tsalo tsalo deleted the nipreps-config branch September 28, 2024 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tsalo tsalo

Labels
maintenance Issues/PRs related to maintenance. Dependency versions, dependencies, etc. refactor Changes to the codebase that don't impact workflow inputs or outputs.
Projects
None yet
Milestone
post-manuscript
Development

Successfully merging this pull request may close these issues.

Adopt configuration approach from nipreps tools
1 participant
@tsalo