Skip to content

Conversation

@Eric-Liu-SANDAG
Copy link
Contributor

@Eric-Liu-SANDAG Eric-Liu-SANDAG commented Dec 30, 2025

Describe this pull request. What changes are being made?

Change the Python package manager from conda to uv.

What issues does this pull request address?

Additional context

New uv environment was tested by running the Estimates Program for 2020 only, results are in database as [run_id] = 122 and [run_id] = 123

Copy link
Contributor

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 migrates the project's Python package management from conda to uv by replacing the environment.yml file with a pyproject.toml file. The change simplifies dependency management by using a modern Python packaging standard while maintaining the same core dependencies.

Key Changes:

  • Removed conda-based environment.yml configuration
  • Added pyproject.toml for uv-based dependency management
  • Specified Python version requirement of >=3.11

Reviewed changes

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

File Description
pyproject.toml New file defining project metadata and dependencies for uv package manager
environment.yml Removed conda environment specification file

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

Copy link
Contributor

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

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.


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

Copy link
Contributor

@GregorSchroeder GregorSchroeder left a comment

Choose a reason for hiding this comment

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

please update README.md to reflect usage of uv, where to download utility, etc...

see https://github.com/SANDAG/Population-Sim for an example

@Eric-Liu-SANDAG
Copy link
Contributor Author

Funny enough, the Population-Sim instructions are not correct for using uv. You should not be manually activating the environment and calling python main.py. Instead, you should just use uv run main.py, which does all the work of activating the environment (as well as a bunch of other checks) for you.

Copy link
Contributor

@GregorSchroeder GregorSchroeder left a comment

Choose a reason for hiding this comment

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

Please make an issue in all our repositories assigning the issues to myself, Bryce, and/or Kevin noting to add you as the reviewer on the subsequent PRs.

Make sure everyone knows they need to update the README.md and remove the environment.yml file in addition to creating the toml and lock files.

@Eric-Liu-SANDAG Eric-Liu-SANDAG merged commit fc7d812 into main Dec 30, 2025
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.

[FEATURE] Change package manager from conda to uv

3 participants