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

Wrappers: Move config accesses to params #513

Open
tedil opened this issue Jun 3, 2024 · 0 comments · May be fixed by #536
Open

Wrappers: Move config accesses to params #513

tedil opened this issue Jun 3, 2024 · 0 comments · May be fixed by #536
Assignees
@tedil @ericblanc20

Comments

@tedil
Copy link
Member

tedil commented Jun 3, 2024

Wrappers should in principle be stand-alone. In our case, this means that they should never access config values directly, but only via snakemake.params.
@tedil tedil self-assigned this Jun 3, 2024
@tedil tedil linked a pull request Aug 22, 2024 that will close this issue
chore: move config accesses in wrappers to params #536
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tedil tedil

@ericblanc20 ericblanc20

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: move config accesses in wrappers to params bihealth/snappy-pipeline
2 participants
@tedil @ericblanc20