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

🐛 Snakemake-minimal error reading config file #2276

Open
enryH opened this issue May 25, 2023 · 5 comments
Open

🐛 Snakemake-minimal error reading config file #2276

enryH opened this issue May 25, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@enryH
Copy link
Contributor

enryH commented May 25, 2023

Snakemake version
snakemake-minimal=7.26.0 installed using conda from bioconda.

snakemake-minimal         7.26.0             pyhdfd78af_0    bioconda

Describe the bug

Using github runners macos-latest, windows-latest and ubuntu-lastest the config file of a workflow
is not read. The introduction of the yte package for reading config files seems to cause the error.

Logs

AttributeError in file /home/runner/work/repo/workflow/Snakefile, line 6:
module 'plac' has no attribute 'flg'
  File "/home/runner/work/work/repo/workflow/Snakefile", line 6, in <module>
  File "/usr/share/miniconda3/envs/pkg/lib/python3.8/site-packages/yte/__init__.py", line 74, in <module>
Error: Process completed with exit code 1.

Minimal example

Additional context

@enryH enryH added the bug Something isn't working label May 25, 2023
enryH added a commit to RasmussenLab/pimms that referenced this issue May 26, 2023
-> configfile loading error after snakemake-minimal update
    to yte, see snakemake/snakemake#2276
@maarten-k
Copy link
Contributor

Can you check the "plac" and "yte" versions in your environment? I think this bug is introduced in #2269

@enryH
Copy link
Contributor Author

enryH commented Jun 4, 2023

Yes sure

plac                      0.9.6                      py_1    conda-forge
yte                       1.5.1              pyha770c72_2    conda-forge

for the ubuntu-latest image.

@maarten-k
Copy link
Contributor

It looks like that fastai, needs spacy, which needs an old version of plac.

Snakemake needs YTE which needs a modern version of plac, but YTE does not restrict the use of a minimal version number of plac This should be considered as a bug.

However, your environment file is quite big and at a certain point all dependencies become a hairball that is unmanageable. The version of programs are not fixed and unwanted behaviour might occur.( features in third-party software might disappear or change) . I advise using the integrated package manager of snakemake to trim down the size of the environment file https://snakemake.readthedocs.io/en/stable/snakefiles/deployment.html#integrated-package-management.

@enryH
Copy link
Contributor Author

enryH commented Jun 5, 2023

That is a fair point... I'll add rule specific dependency files to resolve this issue.

enryH added a commit to RasmussenLab/pimms that referenced this issue May 7, 2024
@enryH
Copy link
Contributor Author

enryH commented May 7, 2024

For me setting a minimal plac version solves the issue ( 1.0).

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

No branches or pull requests

2 participants