Skip to content

add snap tooltip to cursor to see plot values#15

Draft
trns1997 wants to merge 1 commit intoGenesis-Embodied-AI:mainfrom
trns1997:cursor_display
Draft

add snap tooltip to cursor to see plot values#15
trns1997 wants to merge 1 commit intoGenesis-Embodied-AI:mainfrom
trns1997:cursor_display

Conversation

@trns1997
Copy link
Contributor

Not sure if Issue #12 wanted this?

Screenshot from 2025-11-21 16-10-09

@leducp
Copy link
Collaborator

leducp commented Nov 21, 2025

I expected a dedicated side panel but this one is still cool!

@trns1997
Copy link
Contributor Author

ping @leducp


std::string const& name() const { return name_; }
Statistics compute_statistics(double begin, double end) const;
std::vector<Parser::Point> const& get_series() const { return serie_; }
Copy link
Collaborator

Choose a reason for hiding this comment

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

avoid get_ (Qt style)

}
}

if (closestSerie && closestPoint)
Copy link
Collaborator

Choose a reason for hiding this comment

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

snake case

@leducp
Copy link
Collaborator

leducp commented Nov 27, 2025

This code can only works on the downsampled data or small series (< 300k). With a bigger serie, when zooming, the display will switch to the sections to display the full dataset but only on the current span (Level Of Detail approach).

A better approach could be to ask the series to do the rendering localy (like the curve plot): the plot create the tooltip, show mouse x, and each serie draw the nearest point x,y with scatter and color.
With such approach, the serie can switch between downsampled and sections and internal data do not need to be exposed
WDYT?

@trns1997
Copy link
Contributor Author

This code can only works on the downsampled data or small series (< 300k). With a bigger serie, when zooming, the display will switch to the sections to display the full dataset but only on the current span (Level Of Detail approach).

A better approach could be to ask the series to do the rendering localy (like the curve plot): the plot create the tooltip, show mouse x, and each serie draw the nearest point x,y with scatter and color. With such approach, the serie can switch between downsampled and sections and internal data do not need to be exposed WDYT?

ah right clearly didnt try zooming in :/. Alright i will take you up on the approach

@leducp leducp marked this pull request as draft December 16, 2025 17:08
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