-
Notifications
You must be signed in to change notification settings - Fork 250
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
Document how to run Python examples #6218
Conversation
There was documentation there, but I couldn't understand it. Thankfully Jeremy told me the magic incantation, so I decided to write it down later.
Deployed docs
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although that might work, I'm not convinced it's simpler, and it's certainly not how it was intended.
pixi shell | ||
pixi run py-build | ||
pip install examples/python/minimal | ||
minimal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
well that's not really the intended way. All examples are already installed in the examples
environment, which just needs to have a compatible rerun-sdk there, so:
pixi run -e examples py-build
pixi run -e examples minimal
pixi shell | ||
pip install rerun-sdk==0.15.1 | ||
pip install examples/python/minimal | ||
minimal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Likewise.
pixi run -e examples pip install rerun-sdk==0.15.1
pixi run -e examples minimal
To clarify, So both are perfectly fine imo:
and:
The latter case works because |
Closed by #6466 (review) |
There was documentation there, but I couldn't understand it. Thankfully Jeremy told me the magic incantation, so I decided to write it down in a way that I can understand.
I decided to remove the
-e examples
part of it, since everything seems to work without it and because this is the docs for theexamples
environment says it is WIP:rerun/pixi.toml
Lines 39 to 40 in 0ac38a4
Checklist
main
build: rerun.io/viewernightly
build: rerun.io/viewerTo run all checks from
main
, comment on the PR with@rerun-bot full-check
.