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 alloc snakemake kernel #310

Merged
merged 7 commits into from
Dec 19, 2024
Merged

Add alloc snakemake kernel #310

merged 7 commits into from
Dec 19, 2024

Conversation

JosseVanDelm
Copy link
Contributor

This PR adds the basic infrastructure to run a snakemake experiment for the alloc kernel, I will update the other kernels with different PRs :)

Copy link
Contributor

@jorendumoulin jorendumoulin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

"{config[vltsim]} {input[0]}"


use rule * from default_rules exclude compile_simple_main as default_*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is to include/import rules from a different snakemake file declared as a module.
You use the module declaration to choose the file and to apply the configuration, and then those rules act as if they have been included in the file, but with the default_ prefix.
In this case, a lot of rules are shared for many jobs, so instead of defining them everywhere multiple times, I import them from default_rules.smk

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but why exclude compile_simple_main?

@JosseVanDelm JosseVanDelm merged commit db2ea35 into main Dec 19, 2024
17 checks passed
@JosseVanDelm JosseVanDelm deleted the Josse/add-snakemake-kernels branch December 19, 2024 09:54
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