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

Inconsistent behavior of --summary and --detailed-summary #1917

Closed
dariober opened this issue Oct 18, 2022 · 0 comments · Fixed by #2218
Closed

Inconsistent behavior of --summary and --detailed-summary #1917

dariober opened this issue Oct 18, 2022 · 0 comments · Fixed by #2218
Labels
bug Something isn't working

Comments

@dariober
Copy link
Contributor

dariober commented Oct 18, 2022

Snakemake version

7.16.0

Describe the bug

Option --summary gives a cryptic error if you combine it with -j option. Conversely, --detailed-summary gives an error if -j option is missing. I think the expected behavior should be that both --summary and --detailed-summary work with or without -j or at least have the same requirement.

Minimal example

Cryptic error from --summary with -j:

snakemake -j 1 --summary
Building DAG of jobs...
WorkflowErrorin line 1 of /home/dario/Downloads/Snakefile:
Resource _nodes is of type int but global resource constraint defines <class 'str'> with value 1. Resources with the same name need to have the same types (int, float, or str are allowed).

Error from --detailed-summary without -j:

snakemake --detailed-summary
Error: you need to specify the maximum number of CPU cores to be used at the same time. If you want to use N cores, say --cores N or -cN. For all cores on your system (be sure that this is appropriate) use --cores all. For no parallelization use --cores 1 or -c1.

Minimal snakefile:

rule all:
    input:
        ['done.txt', 'foo.txt'],

rule one:
    output:
        '{x}.txt',
@dariober dariober added the bug Something isn't working label Oct 18, 2022
johanneskoester added a commit that referenced this issue Apr 28, 2023
…2218)

### Description

This fixes #1917.

### QC

* [x] The PR contains a test case for the changes or the changes are
already covered by an existing test case.
* [x] The documentation (`docs/`) is updated to reflect the changes or
this is not necessary (e.g. if the change does neither modify the
language nor the behavior or functionalities of Snakemake).

Co-authored-by: Johannes Köster <johannes.koester@tu-dortmund.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant