Deeparg singularity container fix 2 (#1406)

* fix: remove left-over unnecessary code

* Switch to more portable solution for singularity container issue by using bind paths

* Fix input collision of dummy files

* Repalce dummy with which bash

* Remove dummy usage from tests

* Apply suggestions from code review

* Fix singularity typo
This commit is contained in:
James A. Fellows Yates 2022-03-18 16:21:55 +01:00 committed by GitHub
parent ea41a8a6f7
commit 15c7190e22
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
5 changed files with 10 additions and 10 deletions

View file

@ -8,10 +8,10 @@ process DEEPARG_DOWNLOADDATA {
'https://depot.galaxyproject.org/singularity/deeparg:1.0.2--pyhdfd78af_1' : 'https://depot.galaxyproject.org/singularity/deeparg:1.0.2--pyhdfd78af_1' :
'quay.io/biocontainers/deeparg:1.0.2--pyhdfd78af_1' }" 'quay.io/biocontainers/deeparg:1.0.2--pyhdfd78af_1' }"
/* /*
We have to force singularity to run with --fakeroot to allow reading of a problematic file with borked read-write permissions in an upstream dependency (theanos). We have to force singularity to run with -B to allow reading of a problematic file with borked read-write permissions in an upstream dependency (theanos).
This flag may not be available on all systems and may be considered a security problem. so please document and /or warn for this in your pipeline! Original report: https://github.com/nf-core/funcscan/issues/23
*/ */
containerOptions { "${workflow.containerEngine}" == 'singularity' ? '--fakeroot' : '' } containerOptions { "${workflow.containerEngine}" == 'singularity' ? '-B $(which bash):/usr/local/lib/python2.7/site-packages/Theano-0.8.2-py2.7.egg-info/PKG-INFO' : '' }
input: input:

View file

@ -17,7 +17,7 @@ tools:
licence: ["MIT"] licence: ["MIT"]
input: input:
- none: There is no input. This module downloads a pre-built database for use with deepARG. - none: No input required for download module.
output: output:
- versions: - versions:

View file

@ -6,13 +6,13 @@ process DEEPARG_PREDICT {
conda (params.enable_conda ? "bioconda::deeparg=1.0.2" : null) conda (params.enable_conda ? "bioconda::deeparg=1.0.2" : null)
container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ? container "${ workflow.containerEngine == 'singularity' && !task.ext.singularity_pull_docker_container ?
'https://depot.galaxyproject.org/singularity//deeparg:1.0.2--pyhdfd78af_1' : 'https://depot.galaxyproject.org/singularity/deeparg:1.0.2--pyhdfd78af_1' :
'quay.io/biocontainers/deeparg:1.0.2--pyhdfd78af_1' }" 'quay.io/biocontainers/deeparg:1.0.2--pyhdfd78af_1' }"
/* /*
We have to force singularity to run with --fakeroot to allow reading of a problematic file with borked read-write permissions in an upstream dependency (theanos). We have to force singularity to run with -B to allow reading of a problematic file with borked read-write permissions in an upstream dependency (theanos).
This flag may not be available on all systems and may be considered a security problem. so please document and /or warn for this in your pipeline! Original report: https://github.com/nf-core/funcscan/issues/23
*/ */
containerOptions { "${workflow.containerEngine}" == 'singularity' ? '--fakeroot' : '' } containerOptions { "${workflow.containerEngine}" == 'singularity' ? '-B $(which bash):/usr/local/lib/python2.7/site-packages/Theano-0.8.2-py2.7.egg-info/PKG-INFO' : '' }
input: input:
tuple val(meta), path(fasta), val(model) tuple val(meta), path(fasta), val(model)

View file

@ -5,5 +5,5 @@ nextflow.enable.dsl = 2
include { DEEPARG_DOWNLOADDATA } from '../../../../modules/deeparg/downloaddata/main.nf' include { DEEPARG_DOWNLOADDATA } from '../../../../modules/deeparg/downloaddata/main.nf'
workflow test_deeparg_downloaddata { workflow test_deeparg_downloaddata {
DEEPARG_DOWNLOADDATA () DEEPARG_DOWNLOADDATA ( )
} }

View file

@ -13,7 +13,7 @@ workflow test_deeparg_predict {
'LS' 'LS'
] ]
DEEPARG_DOWNLOADDATA() DEEPARG_DOWNLOADDATA( )
DEEPARG_PREDICT ( input, DEEPARG_DOWNLOADDATA.out.db ) DEEPARG_PREDICT ( input, DEEPARG_DOWNLOADDATA.out.db )
} }