Skip to content
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.

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

Pixi Setup (python) #8

Merged
merged 24 commits into from
Jun 20, 2024
Merged

Pixi Setup (python) #8

merged 24 commits into from
Jun 20, 2024

Conversation

alberthli
Copy link
Contributor

@alberthli alberthli commented Jun 18, 2024

This PR sets up pixi as the package manager for obelisk. Sets up the python part, will integrate with c++ part in a separate PR.

Done:

  • added some feature and environment sets, confirmed that it works as expected
  • set up linting, formatting, typechecking, and testing + setting files for all (since there doesn't seem to be a tool section in the pixi.toml). most are registered as pre-commit hooks except the typechecking with pyright
  • added a basic code checking workflow that runs when PRing into main
  • add thin Dockerfile for development that can run pixi shell. also updated the startup.sh file.
  • set up initial CI pipeline, confirmed passing with dummy tests and no package files
  • add ros2 stuff
    • add ros2 dependencies in the pixi.toml
    • add some ros2 code in a workspace to test out the installation
    • set up pixi task for handling overlay installation of the workspace in this repo
  • update README and docs
  • double check whether i can do GPU-compatible tests for CI... maybe i can set up vulcan as a local runner
  • refactor away the startup.sh file to a docker-compose.yml file that works for both the dev container and deployment
  • add manual instructions for the newgrp stuff

Future PRs:

alberthli added 22 commits June 17, 2024 22:46
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
	modified:   README.md

dev_setup.sh
docker
docs
LICENSE
obelisk
obelisk_ws
pixi.lock
pixi.toml
pyrightconfig.json
pytest.ini
README.md
ruff.toml
scripts
tests
dev_setup.sh
docker
docs
LICENSE
obelisk
obelisk_ws
pixi.lock
pixi.toml
pyrightconfig.json
pytest.ini
README.md
ruff.toml
scripts
tests
3laws_amberlab
ambersim
argus
_backup
cache
calibration_data.png
cc_test.py
CSDI
cube_rotation
curobo
drake
drake_playground
DynamixelSDK
example.py
fileback.png
filedown.png
filefront.png
fileleft.png
fileright.png
fileup.png
fr3
frogger
goblet.png
idtx
jaxngp
jaxosqp
jax_simplex
LEAP_Hand_API
mjpc_ros2
ml-agents
mujoco
mujoco_menagerie
mujoco_mpc
nerf_grasping
nerfstudio
nerf_to_mesh.py
obelisk
_old
panda-py
perseus
pfc
pyink
pytorch3d
qwerty_cube_assets.pdf
RCareWorld
rlcbf
rubiks_curvature.png
sdfstudio
sim_script.py
sim_simple_script.py
slides
tapnet
test_grasping
tiny-cuda-nn
torchquad
TorchSDF
torch-template-repo
tsdf_refactor
urdf-viz
vince_ros
vince_ros.zip
dev_setup.sh
docker
docs
LICENSE
obelisk
obelisk_ws
pixi.lock
pixi.toml
pyrightconfig.json
pytest.ini
README.md
ruff.toml
scripts
tests
dev_setup.sh
docker
docs
LICENSE
obelisk
obelisk_ws
pixi.lock
pixi.toml
pyrightconfig.json
pytest.ini
README.md
ruff.toml
scripts
tests
On branch pixi-setup
Changes to be committed:
  (use "git restore --staged <file>..." to unstage)
	modified:   README.md

3laws_amberlab
ambersim
argus
_backup
cache
calibration_data.png
cc_test.py
CSDI
cube_rotation
curobo
drake
drake_playground
DynamixelSDK
example.py
fileback.png
filedown.png
filefront.png
fileleft.png
fileright.png
fileup.png
fr3
frogger
goblet.png
idtx
jaxngp
jaxosqp
jax_simplex
LEAP_Hand_API
mjpc_ros2
ml-agents
mujoco
mujoco_menagerie
mujoco_mpc
nerf_grasping
nerfstudio
nerf_to_mesh.py
obelisk
_old
panda-py
perseus
pfc
pyink
pytorch3d
qwerty_cube_assets.pdf
RCareWorld
rlcbf
rubiks_curvature.png
sdfstudio
sim_script.py
sim_simple_script.py
slides
tapnet
test_grasping
tiny-cuda-nn
torchquad
TorchSDF
torch-template-repo
tsdf_refactor
urdf-viz
vince_ros
vince_ros.zip after the dev_setup.sh script
@alberthli alberthli requested a review from Zolkin1 June 20, 2024 06:20
@alberthli alberthli marked this pull request as ready for review June 20, 2024 06:20
Copy link
Collaborator

@Zolkin1 Zolkin1 left a comment

Choose a reason for hiding this comment

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

Overall lgtm. A few small comments and some questions, but should be able to merge in very soon.

Copy link
Collaborator

@Zolkin1 Zolkin1 left a comment

Choose a reason for hiding this comment

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

lgtm

@Zolkin1 Zolkin1 merged commit 8da79aa into main Jun 20, 2024
1 check passed
@alberthli alberthli deleted the pixi-setup branch June 30, 2024 07:43
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.

2 participants