-
Notifications
You must be signed in to change notification settings - Fork 11
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? #17
Comments
Or better yet, would you consider adding/merging this to nf-core (https://nf-co.re/)? |
Hi @AntonioBaeza and @mahesh-panchal, First of all, thanks for your interest in the pipeline. about condaYes, updating the pipeline to also use conda environments instead of only Docker is definitely in my TODO/whish list. However, I was not able to fully work on it yet because some of the tools are not yet available in conda. I am not very much of an expert in conda package development. I am still studying on how to help on that and create a bioconda package for the tools that yet don't have it. And a help on that would be deeply appreciated if any of you have some knowledge on it, or know someone that would like to collaborate. about nf-coreAbout the nf-core consideration, I would gladly merge the pipeline into nf-core. However I have two considerations on it:
I will try to participate on the hackathon that you mentioned to see if the community together can help on solving this problems and work together on the best solution for that: either merging it in nf-core or developing new bioconda packages so that the pipeline can also run with conda-only. In the mean time, do you know where I could starting discussing it with the nf-core community and developers? Thanks again for your interests and ideas. Any help is immensely welcomed. |
Re: nf-core Please join us on slack and discuss this in the #new-pipelines channel (https://nf-co.re/join/slack). |
I have re-checked the tools in the bioconda channel and it now seems that all the tools are available there. Thus, in the meantime, I will start re-shaping the pipeline so it can either run with Docker and conda as suggested by @AntonioBaeza. Re: nf-core Thank you again @mahesh-panchal for the information. I will bring the suggestion and discussion to our lab. group so we can then start discussions with nf-core community. |
Adding to nf-core would be really great! I was about to ask for the possibility to use the pipeline with sge and singularity. Both features come out of the box with nf-core. |
In theory, both capabilities should be possible because it just depends on adjusting the nextflow config file to understand the profiles sge and singularity. But definitely it would be easier if added to nf-core. The only problem is setting the processes to use conda and transforming the pipeline into the nf-core format. I'll definitely need help on this. I forgot to do it before, I will ask in the nf-core slack channel whether they're interested in adding it. |
I will just finish the implementation of the multi-sample workflow that is described in #15 before that. This is very near to it's release and when done. |
Now that the pipeline has been reconfigured to understand a YAML so it can analyse multiple samples at once. I will start to take a look on the possibility to set the modules to use bioconda channels and bioconda docker images so it has an easy integration for using conda, docker or singularity. This will be tackled in branch https://github.com/fmalmeida/MpGAP/tree/issue-17-conda |
Made some tests with a simpler pipeline (https://github.com/fmalmeida/ngs-preprocess) and was able to make it run with docker, conda and singularity. I will try then to implement this possibility in MpGAP using what I've learnt from the implementation in this other pipeline, focusing on using bioconda packages. |
Obrigado Felipe!!!
…On Mon, Nov 8, 2021 at 4:16 PM Felipe Marques de Almeida < ***@***.***> wrote:
Made some tests with a simpler pipeline (
https://github.com/fmalmeida/ngs-preprocess) and was able to make it run
with docker, conda and singularity.
I will try then to implement this possibility in MpGAP using what I've
learnt from the implementation in this other pipeline, focusing on using
bioconda packages.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#17 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALIJRA5E72XWF774OFL2TDTULA42RANCNFSM5EJT2Q2A>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
J. Antonio Baeza
Associate Professor
Department of Biological Sciences, Clemson University
South Carolina, USA
&
Research Associate
Smithsonian Marine Station at Fort Pierce, Florida, USA
&
Adjunct Faculty
Universidad Catolica del Norte, Coquimbo, Chile
Email: ***@***.*** & ***@***.***
Website: http://baezaantonio.wix.com/baezalabclemson
Website (CI-team): https://baezaantonio.wixsite.com/clemsonmitogenomics
Website ResearchGate: https://www.researchgate.net/profile/J_Baeza/
Website SemanticScholar:
https://www.semanticscholar.org/author/Juan-Antonio-Baeza/144723920
|
Hello guys, Just to update that I tried to address this issue in PR #25. This is the first time that I try to implement my pipelines with either conda and singularity at once, moreover, some tools used in it are either only available for linux-64, or its conda packages are neither available nor up-to-date, causing incompatibility with other tools. Thus, to solve the tools that had this problem, I have made pre-compiled linux-64 binaries of them available inside the main/root pipeline bin directory. Therefore, the Please, since it is the first time I did it, the Read more at: https://github.com/fmalmeida/MpGAP#selecting-between-profiles 😄 |
Looks like a great pipeline. Any chance you can have it in conda?
The text was updated successfully, but these errors were encountered: