Skip to content

Conversation

@gsfaigan
Copy link

@gsfaigan gsfaigan commented Sep 24, 2025

Purpose

Closes #340.
Consolidate the Python project setup files into a single pyproject.toml to simplify package management. Removed setup.py, setup.cfg, and requirements.txt as their contents are now fully managed in pyproject.toml. This reduces redundancy and aligns with modern Python packaging standards.

New Changes

  • Migrated all dependencies from requirements.txt to pyproject.toml.
  • Consolidated setup.py and setup.cfg configurations into pyproject.toml.
  • Explicitly defined Python packages: gs, gs/backend, obc/tools/python, and interfaces.
  • Added py.typed markers for proper type hinting support.
  • Removed unused top-level directories (hil, libs, cmake, python_test) from package discovery to prevent setuptools errors.

Testing

Explain tests that you ran to verify code functionality.

  • Verified python3 -m pip install .[dev] installs successfully in a fresh virtual environment without errors. THIS DOESN'T WORK RIGHT NOW
  • Verified dev dependencies install correctly from pyproject.toml.
  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested (setup changes only, no functional code modified).
  • I have tested this PR on a board if the code will run on a board (Only required for firmware developers).
  • I have tested this PR by running the ARO website (Only required if the code will impact the ARO website).
  • I have tested this PR by running the MCC website (Only required if the code will impact the MCC website).
  • I have included screenshots of the tests performed below.

Outstanding Changes

  • Could migrate to a src/ layout in the future for even cleaner package organization.
  • Additional verification of optional dev dependencies to ensure full compatibility.

@github-actions
Copy link

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K 24 20h 4m 92
kepler452b123 17 4d 1h 1m 74
camspec 10 17h 17m 68
Yarik-Popov 8 1d 4h 42m 65
Navtajh04 6 5d 1h 23m 68

⚡️ Pull request stats

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.

Clean up Python setup

1 participant