vis compare flag + uv environment change + fixing Github workflow - #73
Merged
Conversation
vis compare flag + uv changes + git workflow
harshi-puli
marked this pull request as ready for review
August 17, 2026 01:47
added default algo logic + label settings
Added new default algo alerting system
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
vis.py
--comparemode, uv migration, CI re-enableSummary
vis.py: new--compare <algo>flag. Runs a second registered algo for the same--taskon identical frames and stacks its debug-frame grid below the primary algo's in one "Debug Frames" window (not beside it, so neither pane's width shrinks). Each half is corner-labeled with its algo name; sliders for both algos share the window, prefixed by algo name (e.g.center: canny_low) so they don't collide. You can opt to hide labels using the --hide_labels flagsetup.pyreplaced withpyproject.toml; README installation instructions rewritten accordingly..github/workflows/ci.yml): restoredpush/pull_requesttriggers. The error before was that it was breaking on the old slot machine test file so that was renamed. Also got rid of the lint error error check due too there being too many to fix at the moment.How to test
--compareq/Escquits,ppauses,i/oslow down/speed up playback.slalomonly has one registered algo (classical) right now, so there's nothing to compare it against yet.uv setup vs. the old conda flow
curl -LsSf https://astral.sh/uv/install.sh | sh(orbrew install uv)conda create -n urobotics python=3.11uv venv --python 3.11(downloads 3.11 automatically if missing, creates.venv/in the repo)conda activate uroboticssource .venv/bin/activatepip3 install -e ./uv pip install -e .pip3 install -r requirements-classical.txtuv pip install -r requirements-classical.txtpip3 install -r requirements-torch.txtuv pip install -r requirements-torch.txtEverything downstream (Cython build,
python -m perception.vis.vis ...,pytest) is unchanged once the venv is activated. Env is now per-repo (.venv/insideperception/, gitignored) rather than one shared named conda env.CI workflow change
Before:
on: workflow_dispatchonly (manual-trigger-only,lint+test-classicaljobs, both effectively dead since nothing ran automatically).After:
on: push / pull_request / workflow_dispatch(runs automatically again),lintjob removed,test-classicaljob unchanged. Verified locally end-to-end (submodule fetch, dependency install, Cython rebuild,pytest -v --tb=short) — 20 passed, 0 failed, only expected torch-not-installed skip warnings.