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

fix: use conda as the default frontend #2547

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

victorlin
Copy link

Description

With conda's October 2023 release¹, the default solver uses the same backend as mamba, making it equally fast.

¹ https://github.com/conda/conda/blob/f9fe520956490f7d218abbe6f6fdc0c361535847/CHANGELOG.md#23100-2023-10-30

QC

This PR has been opened for discussion. Further changes can be made if this seems like a good direction.

  • The PR contains a test case for the changes or the changes are already covered by an existing test case.
  • The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

With conda's October 2023 release¹, the default solver uses the same backend as mamba, making it equally fast.

¹ https://github.com/conda/conda/blob/f9fe520956490f7d218abbe6f6fdc0c361535847/CHANGELOG.md#23100-2023-10-30
Copy link

sonarcloud bot commented Dec 19, 2023

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@johanneskoester johanneskoester changed the title Use conda as the default frontend fix: use conda as the default frontend Dec 20, 2023
@BEFH
Copy link

BEFH commented Feb 5, 2024

Are you sure they are equally fast? The solver isn't the only requirement for building environments

@victorlin
Copy link
Author

@BEFH No, I haven't done any benchmarking. I imagine it's comparable though. The benefit would be one less runtime dependency (mamba).

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 this pull request may close these issues.

None yet

2 participants