-
Notifications
You must be signed in to change notification settings - Fork 1
Jay/gen 977 add param sliders #29
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
Conversation
|
@sharlaon Nice! What are the UI/UX changes here so that I can fire up the notebook and give it a spin? I'm trying to answer my own question by inferring from the diffs, but that's probably not a great approach 😅 |
|
@eightysteele Yeah so now the widgets all have sliders for adjusting the sensor noise parameters. Some widgets get two: one for the data snapshot, plus one for the assumption the model is using when working on that data to infer pose. |
|
(The other changes are just those required to make this possible.) |
|
@sharlaon Got it, thanks! For this visualization, what's expected when I move the slider? Nothing changes for me:
|
eightysteele
left a comment
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.
@sharlaon lgtm, although some of the visualizations don't appear to respond to slider changes.
|
@eightysteele Really?! For me there is definitely motion in response to the slider on that widget. |

No description provided.