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

Update resources handler to allow empty resource specifications #75

Open
sorelfitzgibbon opened this issue Dec 15, 2024 · 1 comment
Open

Comments

@sorelfitzgibbon
Copy link
Contributor

I (or rather Claude) have written python scripts to convert from json to tsv and back again. In converting back to json processes without specification (many are in f2) are still included with empty brackets, examples below. I (Claude) could easily update to fix this, but it might be nice to have the same set of processes specified for each node with empty brackets showing that it's not defined.

        "call_sSNV_Strelka2": {
            "cpus": {
                "min": 2,
                "fraction": 1,
                "max": 2
            },
            "memory": {}
        },
        "call_sSNV_Mutect2": {
            "cpus": {},
            "memory": {}
        },
@yashpatel6
Copy link
Collaborator

This is worth discussing in NF WG though I'm not a big fan of leaving undefined items in the allocations as it ends up accumulating and requires specific logic to be maintained to handle them along with Nextflow's default behaviors

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

No branches or pull requests

2 participants