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

r-base is (sometimes) a dependency for picard #32

Closed
florianzwagemaker opened this issue May 6, 2019 · 8 comments
Closed

r-base is (sometimes) a dependency for picard #32

florianzwagemaker opened this issue May 6, 2019 · 8 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@florianzwagemaker
Copy link
Collaborator

r-base needs to be added as a dependency to the scaffold_analysis.yaml conda template.

specifc version is r-base==3.5.1 (latest)

related to bioconda/bioconda-recipes#3859

@florianzwagemaker florianzwagemaker added the bug Something isn't working label May 6, 2019
@florianzwagemaker florianzwagemaker added this to the v0.9.2 milestone May 6, 2019
@florianzwagemaker florianzwagemaker self-assigned this May 6, 2019
@florianzwagemaker
Copy link
Collaborator Author

Should be solved in DennisSchmitz/jovian@548068e

@thierryjanssens
Copy link
Collaborator

I have a similar experience that with certain samples Picard requires the ldpaths package:

ERROR 2019-05-10 11:27:12 ProcessExecutor /data/BioGrid/ERVINGS/Runs_Respiratory_MiSEQ_RIVM/MiSeq_RUN_12APR2019/Jovian/.snakemake/conda/1bd8fc38/lib/R/bin/R: line 238: /data/BioGrid/ERVINGS/Runs_Respiratory_MiSEQ_RIVM/MiSeq_RUN_12APR2019/Jovian/.snakemake/conda/1bd8fc38/lib/R/etc/ldpaths: No such file or directory
[Fri May 10 11:27:12 CEST 2019] picard.analysis.CollectInsertSizeMetrics done. Elapsed time: 0.07 minutes.
Runtime.totalMemory()=536870912
To get help, see http://broadinstitute.github.io/picard/index.html#GettingHelp
Exception in thread "main" picard.PicardException: R script picard/analysis/insertSizeHistogram.R failed with return code 1
at picard.analysis.CollectInsertSizeMetrics.finish(CollectInsertSizeMetrics.java:184)
at picard.analysis.SinglePassSamProgram.makeItSo(SinglePassSamProgram.java:164)
at picard.analysis.SinglePassSamProgram.doWork(SinglePassSamProgram.java:84)
at picard.cmdline.CommandLineProgram.instanceMain(CommandLineProgram.java:295)
at picard.cmdline.PicardCommandLine.instanceMain(PicardCommandLine.java:103)
at picard.cmdline.PicardCommandLine.main(PicardCommandLine.java:113)

@thierryjanssens
Copy link
Collaborator

Perhaps ldpaths needs to be added to the scaffold_analysis.yaml conda template as well?

@thierryjanssens
Copy link
Collaborator

Forgot to re-open.

@thierryjanssens
Copy link
Collaborator

A similar case has been observed in the following issue:

conda-forge/r-base-feedstock#67

@florianzwagemaker
Copy link
Collaborator Author

ldpaths is unfortunately not listed in conda as an independent package. Ldpaths as an issue is unfortunately out of our hands for that matter.
It however seems that deleting the particular conda environment and re-running the pipeline can help. The conda environment should be re-made when the pipeline starts.

will keep an eye out on the r-base issue you linked. Once published we can test it and if it indeed solves the issue then we should update the environment dependencies.

@DennisSchmitz
Copy link
Owner

Just thinking out loud here, haven't thoroughly read the linked r-base issue... But, would Singularity help with this?

@florianzwagemaker
Copy link
Collaborator Author

Closing for now,
Ldpaths itself is an issue related to

  1. The conda-forge package
  2. The filesystem latency

If you run into this issue then the best thing you can do is to simply rebuild the specific conda environment.

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

3 participants