-
Notifications
You must be signed in to change notification settings - Fork 96
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
Added filtering and other options to SampleView of Databrowser #2840
base: master
Are you sure you want to change the base?
Added filtering and other options to SampleView of Databrowser #2840
Conversation
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.
Works as described, but I have some comments on usability:
- The "Move Samples View Up/Down" feels a bit superfluous. After all, you can drag the divider to set the vertical size of the samples view.
- To set/unset/change filter you need to switch between samples view and properties view. It would be easier if filter controls would be added to the samples view. Since user selects a trace in the dropdown, the filter UI controls could be updated when selection changes (and disabled when user selects "all"). Also, if the filter selection is moved to the samples view, an update can be triggered automatically, currently user needs to click Refresh.
- Filters "Threshold Up" and "Threshold Change" overlap a bit. Is "Threshold Down" not a relevant use case?
Thank you for the comment. Having the Filters in the Sample View is a good idea. In our use-case it would be helpful to to be able to view the Sample View in addition to the Properties. Our goal was to have a kind of log view of archiver data as an alternate view to the plot. I'll work on implementing the rest of the notes and try to think of a more elegant approach for the sample view, since we would prefer to have that as an option. |
fixed NPE when adding PV through sampleview filter UI syncs between views threshold text changed added tooltips to filter controls undo support in SampleView
@thomas1028, sorry for slow feed-back. I've had a discussion with another developer about this feature. Our joint conclusion is that we should maybe consider a separate Phoebus application that would be designed around inspecting and analyzing archiver data. Putting the filters in the samples view of the Data Browser feels a bit odd as filters do not affect the plots, only the table of samples. After all, plotting PV values is the primary purpose of the Data Browser. Any thoughts? |
I like the idea of keeping the data browser as a tool to plot samples over time. |
This PR includes multiple changes to the SampleView of the Databrowser.
SampleView contextmenu can move it's pane up or down if someone prefers to view the samples and property tab simultaneously.
Item selection includes option for "All Items", which displays the samples of all visible PVItems. Selecting this option also adds a "PV" Column to differentiate between the sources
Additional Columns in the Properties Tab for setting a kind of Filter and a value for it.
This allows users to only view samples that fulfil a condition.
The Threshold filter types can be applied to all numeric PVItems, including enums. For enums it needs the index of the desired enum value.
Samples can be hidden through the filter or by hiding the source item. Hiding the source item only hides samples when viewing samples from All Items.