euronion / trace

16 stars 3 forks source link

New update in conflict with README #14

Open MurkyTheMurloc opened 2 years ago

MurkyTheMurloc commented 2 years ago

Hi, with the updated results.smk, it seems like "Solving a scenario" in README doesn't work anymore: snakemake -jall --restart-times 3 results/<scenario>/results.csv

rule all_scenario_results:
    input:
        results=expand(
            "results/{instances}/results.csv",
            instances=custom_instance_pattern(scenarios),
        ),
    output:
        results="results/results.csv",
euronion commented 2 years ago

I'll have a look. The preferred way of running scenarios should be the new scenario management and snakemake -call --restart-times 3 all_scenario_results . I'll update the README.md.

MurkyTheMurloc commented 2 years ago

If I run snakemake -jall --restart-times 3 results/default/results.csv, the error is:

MissingRuleException:
No rule to produce results/default/results.csv (if you use input functions make sure that they don't raise unexpected exceptions).
euronion commented 2 years ago

Yes, also linked to the new scenario management. Use

snakemake -jall --restart-times 3 results/results.csv

instead.

I'll have to do an update to the documentation.