Skip to content

ISSUE 534: Deprecate conda environment.yml / move to pip#552

Open
HundredBillion wants to merge 12 commits intosdatkinson:mainfrom
HundredBillion:issue-534
Open

ISSUE 534: Deprecate conda environment.yml / move to pip#552
HundredBillion wants to merge 12 commits intosdatkinson:mainfrom
HundredBillion:issue-534

Conversation

@HundredBillion
Copy link
Copy Markdown

@HundredBillion HundredBillion commented Apr 26, 2025

This PR is meant to resolve Issue 534.

Since PyTorch is no longer shipping conda packages, I updated this repo to remove the conda environment .yml files and switch the installation docs over to a pip / virtualenv-based setup.

For GPU installs, PyTorch’s commands depend on the user’s OS and CUDA version, so instead of hardcoding those here, the docs now point to the official PyTorch install page:

https://pytorch.org/get-started/locally/

After PyTorch is installed, NAM can be installed with pip.

Summary of changes

  • removed environments/environment_cpu_apple.yml
  • removed environments/environment_gpu.yml
  • removed requirements-gpu.txt
  • updated docs/source/installation.rst to describe a pip/venv-based install flow

@HundredBillion HundredBillion marked this pull request as ready for review April 26, 2025 18:37
@HundredBillion HundredBillion changed the title Issue 534 Issue 534: Deprecate conda environment.yml / move to pip Apr 26, 2025
@HundredBillion HundredBillion changed the title Issue 534: Deprecate conda environment.yml / move to pip ISSUE 534: Deprecate conda environment.yml / move to pip Apr 26, 2025
@sdatkinson
Copy link
Copy Markdown
Owner

Thanks for the PR, @HundredBillion!

This doesn't resolve #534 because installing still uses conda .yml files.

I'll give comments inline presently

Comment thread docs/source/installation.rst Outdated
Comment thread environments/environment_cpu_apple.yml
Comment thread environments/environment_gpu.yml Outdated
Comment thread environments/environment_gpu.yml Outdated
Comment thread environments/environment_cpu_apple.yml Outdated
Comment thread docs/source/installation.rst Outdated
Comment thread environments/environment_gpu.yml Outdated
@HundredBillion
Copy link
Copy Markdown
Author

@sdatkinson - I noticed your PR to change the url for installing Pytorch.

I updated this PR in case you still want to use update the docs to install via requirements-gpu.txt

Copy link
Copy Markdown
Owner

@sdatkinson sdatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry, but this still isn't what I want. There are yml files still in the repo. Moving to pip should see them gone.

@HundredBillion
Copy link
Copy Markdown
Author

I'm sorry, but this still isn't what I want. There are yml files still in the repo. Moving to pip should see them gone.

Got it — I misunderstood the issue. I treated it as “use pip inside the conda env files,” but what you want is to remove the conda env approach entirely. I’ll update the PR to delete the environment .yml files and switch the installation docs to a pip/venv-based flow.

Copilot AI review requested due to automatic review settings April 19, 2026 20:35
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project’s installation guidance to move away from conda-based environment setup and toward pip/venv-based installation, addressing Issue #534 about PyTorch no longer being available via conda.

Changes:

  • Removed conda environment definitions for CPU (Apple) and GPU setups.
  • Rewrote the installation docs to use Python + venv + python -m pip.
  • Updated local development setup instructions to use editable installs and manual dev tooling installation.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
environments/environment_gpu.yml Removes the GPU conda environment definition.
environments/environment_cpu_apple.yml Removes the Apple/CPU conda environment definition.
docs/source/installation.rst Updates installation and development instructions to use venv + pip and points users to PyTorch’s install selector.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/source/installation.rst Outdated
Comment thread docs/source/installation.rst Outdated
Comment thread docs/source/installation.rst
Comment thread docs/source/installation.rst Outdated
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.

Deprecate conda environment.yml / move to pip

3 participants