Remove conda option for installation#2154
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the repository’s conda-based installation path and updates CI and documentation to treat pip/container workflows as the supported ways to install and develop simtools. In the broader codebase, this reduces maintenance around packaging and test environments while keeping the PyPI and container distribution paths intact.
Changes:
- Delete the root
environment.ymlconda environment definition. - Simplify unit-test CI to use only
actions/setup-pythonwith pip-based installation across the Python version matrix. - Remove conda references from user/developer docs, release docs, README badges, and the changelog.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
environment.yml |
Removes the conda/mamba development environment file. |
docs/source/user-guide/getting_started.md |
Updates end-user installation docs to document pip only. |
docs/source/developer-guide/release.md |
Removes conda feedstock release-maintenance instructions. |
docs/source/developer-guide/getting_started_as_developer.md |
Rewrites local developer setup to use venv instead of conda. |
docs/source/developer-guide/developer_guidelines.md |
Updates dependency-management guidance to reference only pyproject.toml. |
docs/changes/2154.maintenance.md |
Adds changelog fragment for removing conda installation. |
README.md |
Removes the conda package badge. |
.github/workflows/CI-unittests.yml |
Drops mamba-based unit-test setup and runs all matrix jobs via pip. |
.github/copilot-instructions.md |
Updates repository instructions to remove conda-based setup guidance. |
|
orelgueta
approved these changes
May 4, 2026
EshitaJoshi
approved these changes
May 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Simtools allows for two different installation paths using pip or conda. This is nice, but the duplication introduces non-negligible additional work (e.g., the additional updates required on conda forge; failing tests due to slower dependencies updates; additional code). I simply think we cannot maintain it.
This PR removes the conda installation path, as this is the one not required, and its maintenance is more complex than pip.
This should solve the issues with failing unit tests of the past week.