Bug 485 - PV should show the sampling point(s) graphically
Summary: PV should show the sampling point(s) graphically
Status: RESOLVED FIXED
Alias: None
Product: PulseView
Classification: Unclassified
Component: Data display (show other bugs)
Version: unreleased development snapshot
Hardware: All All
: Normal minor
Target Milestone: ---
Assignee: Nobody
URL:
Keywords:
Depends on: 590
Blocks:
  Show dependency treegraph
 
Reported: 2014-11-21 15:15 CET by Uwe Hermann
Modified: 2017-03-18 16:03 CET (History)
1 user (show)



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Uwe Hermann 2014-11-21 15:15:43 CET
It would be nice if PulseView could have a hotkey to toggle the display of tiny little "dots" or some other graphical thing that shows where the actual sampling point of digital (and analog) samples was. This can be quite useful to enable at times to figure out where exactly the digital sample points are in relation to each other, in related to PD annotations, in relation to analog sampling points. etc. etc.
Comment 1 Uwe Hermann 2014-11-21 15:28:00 CET
Another, additional idea from IRC: Have the hotkey toggle between three different states:

Per default only the graph is shown (not "dots", i.e. sampling point markers).

Press the hotkey once -> graph + dots are shown.
Press the hotkey again -> only dots (no graph) are shown.
Press the hotkey again -> we're back to the default of only showing the graph (without dots).
Comment 2 Soeren Apel 2015-06-10 15:43:04 CEST
Instead of having a global mode that changes the display mode for all channels, I'd vote for being able to do this on a per-channel basis. The use case is that such markers could mess up the display of high-frequency signals (e.g. a clock) that the user may wish to correlate to a signal of lower frequency (where the dots would be useful to see).
In this case, having a "reset all" hotkey would be useful as well.
Comment 3 Uwe Hermann 2015-06-13 13:15:54 CEST
Yup, good point. Having it overridable per-channel can indeed be very useful.

Maybe the default could be that the setting applies to all channels, and there's some additional way to also specify it per-channel.
Comment 4 Uwe Hermann 2017-03-18 16:03:37 CET
This is fixed in:

e9c8e87cf09ad9761bea1193b25d01a87f8e0dd6
051ba3b399abbd83fea8b7ef6f8876884ac0786c
8de1e1b2fb1bcaaa21f08b7bba412b0839d6c4d2