Skip to content

migrate to uv, add nix flake #59

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.

Sign up for GitHub

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

Jump to bottom
Draft
gwennlbh wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

migrate to uv, add nix flake #59

gwennlbh wants to merge 6 commits into from

Conversation

@gwennlbh
Copy link

@gwennlbh gwennlbh commented May 22, 2025 •
edited
Loading

Description

Migrate from setup.py to a PEP 621 pyproject.toml file along uv for dependency management

How I Tested

(not tested yet)

Do not merge before

I fulfilled the following requirements

  • All new code is formatted according to our style guide (for C++ run clang-format, for Python, run flake8 and fix all warnings).
  • All new functions/classes are documented and existing documentation is updated according to changes.
  • No commented code from testing/debugging is kept (unless there is a good reason to keep it).

gwennlbh
gwennlbh commented May 22, 2025
View reviewed changes
flake.nix

nativeBuildInputs = [ pkgs.python3Packages.setuptools pkgs.rosPackages.rolling.xacro ];

buildInputs = with pkgs.python3Packages; [ meshcat pybullet ];
Copy link
Author

@gwennlbh gwennlbh May 22, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

TODO: bullet_utils

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@gwennlbh