-
Notifications
You must be signed in to change notification settings - Fork 21
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
conda recepies #62
Comments
Hi @apeltzer, I was away at a conference last week but have got around to creating PRs to add tools to bioconda. Have you permission to review and merge the above PRs? Two areas of concern are detailed above, any guidance here would be greatly appreciated |
I have |
Hi @BarryDigby - just checked (sorry, super busy time right now at work): It looks quite good at the moment to me, e.g. targetscan biocontainer is here: https://quay.io/repository/biocontainers/targetscan |
After x-checking, it seems not to have been uploaded - we should maybe ask in their Support Channel. I also opened an issue on bioconda to ask if something was odd there. See here: bioconda/bioconda-recipes#38099 |
@apeltzer Really appreciate you taking the time to check Alex. The conversion to DSL2 is going pretty smoothly, it's just the containers that are holding up the show (and linting, probably 🤖 ). Trying my best to have it ready for review next week |
@apeltzer Hey Alex, the workflow has been successfully converted to DSL2, the only catch being I am still waiting on those bioconda containers, and for a reason I cannot pin down, singularity fails to pull this image from quay despite it working perfectly fine for Docker. Irritating to say the least 😁 Given the deadline for the response to authors for BMC is December 5th... what's the gameplan here? |
Just asked on the bioconda chat to unhide these & also asked whats going on with Singularity. |
Singularity there in << 24hours, the quay.io should already be there. So "just" in time :-D |
nf-core/circrna
modules that require package builds for the conda repository:CIRIquant
PR: bioconda/bioconda-recipes#38029
Not working because
argparse>=1.2.1
andscipy==1.2.2
are not found using defaults/conda-forge/bioconda.argparse>=1.2.1
is available from this channel:pdrops::argparse==1.2.2
but I cannot figure out how to tell my build to incorporate this channel. I have added it to my local conda config, to no avail...scipy==1.2.2
does not exist on conda.I have tried both pypi skeletons and generic noarch python skeletons here, so I am not sure how to proceed.
circtools
PR: bioconda/bioconda-recipes#37786
posterity..
PR: bioconda/bioconda-recipes#37786
The author of DCC made a pypi package called
circtools
available at https://pypi.org/project/circtools/. This is convenient for me as it bundlesDCC
andCircTest
together, both of which are used in the workflow.Successfully builds, pushed to anaconda to test
TODO: This package works as expected, but I need it to be hosted by
bioconda
and not b.digbyhttps://anaconda.org/b.digby/circtools
bioconda-utils build --docker --mulled-test --packages recipes/circtools
fails, see below:error log
find_circ
PR: bioconda/bioconda-recipes#37923
circrna_finder
PR: bioconda/bioconda-recipes#37922
targetscan
PR: bioconda/bioconda-recipes#37960
I cannot find a license file for targetscan, and it is denoted as being
Copyright (c) The Whitehead Institute of Biomedical Research
.Is it appropriate to add this to bioconda? If not, how can I proceed to ensure I am following nf-core best practices?
DEA
Mulled container for differential exp scripts. PR: BioContainers/multi-package-containers#2382
The text was updated successfully, but these errors were encountered: