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

recommended way to build conda envs #218

Open
wwood opened this issue Oct 30, 2024 · 2 comments · Fixed by #222
Open

recommended way to build conda envs #218

wwood opened this issue Oct 30, 2024 · 2 comments · Fixed by #222

Comments

@wwood
Copy link
Collaborator

wwood commented Oct 30, 2024

Hi,

Kinda newbie thread of thought here.

I'm wanting to run several aviary runs in a snakefile in parallel, but don't want them all building conda envs y themselves. What is the recommendation for a separate step that builds the conda envs? Does aviary recover --build build everything necessary if the argument to build is missing?

I guess I'm looking for something like this to work

aviary recover --conda-prefix ~/e/aviary-conda-prefix --build

or specificalyl for short reads

aviary recover -1 dummy.fastq -2 dummy.fastq --build --conda-prefix ~/e/aviary-conda-prefix

But those both error out with missing input exceptions. Maybe --build should have a default?

Oh, it works when the input read files actually exist (maybe needs to handle that more gracefully).

Also is a bit confusing that there's a [BUILD] afterwards in the help. I interpreted that to mean a specific snakemake rule needs specifying. Maybe [yes|no]?

What about changing the help to

Build conda environments necessary to run the pipeline, and then exit. Equivalent to "--snakemake-cmds '--conda-create-envs-only True ' ". Other inputs should be specified as if running normally so that the right set of conda environments is built.

?

@rhysnewell
Copy link
Owner

Yeah, ideally the build command would just build all of the environments without needing the read files but I unfortunately couldn't get snakemake to co-operate in that regard. The help message can be updated

@AroneyS
Copy link
Collaborator

AroneyS commented Nov 7, 2024

I just used placeholder files to get Aviary conda env built in Bin Chicken (https://github.com/AroneyS/binchicken/blob/823bd135a945142cfb7c26d951616068d8e45e32/binchicken/binchicken.py#L1006)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants