Comparer: click metric graph to seek to that frame (#8)#41
Merged
Conversation
Add MetricCal::FrameAtX that maps a graph-area X pixel to a frame ID under the current view range, and let FrmsInfoView::OnLButtonDown use it to call SetScenes() and refresh the per-frame metric label. Combined with the recently-landed wheel zoom, the user can now zoom into a region of interest and click any point to jump straight to that frame. Double-click still resets the view, and clicks outside the graph area fall through to the default handler.
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.
Closes #8.
The PSNR/SSIM line graph in
FrmsInfoViewnow responds to left-clicks: clicking any point in the graph area callsSetScenes()to set both panes to the frame at that X position, then refreshes the per-frame metric label viaCalMetrics(). The frame is computed against the current view range, so the wheel-zoom and click-to-seek features stack naturally — zoom in to a region of interest and click any point to land exactly on that frame.How to try