-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Aggregate LAD Table view doesn't work properly when children are aggregates #5939
Comments
What about if we used used the LAD Table Set functionality here? ie grouped the children by aggregate parent? We would have to figure out how to handle nesting - that might require a change to the LAD Table Set view (LAD Sets within LAD Sets) |
After discussion with @charlesh88 & @akhenry, we're going to show aggregates that have aggregates (i.e., not leaf telemetry nodes) as a special child, that clicking it will navigate you to. The child will not have a timestamp or value, and may have a "Type" as a "Aggregate". For extra credit, other children (i.e., leaf telemetry nodes) will also be clickable and navigable. |
This issue is related to akhenry/openmct-yamcs#281. |
@charlesh88 and/or @akhenry If the telemetry object is not an "Aggregate", how specific do we want to make the type? Is "Simple" or "Leaf" enough? Or do we want something more specific like "Numeric"? |
@charlesh88 @akhenry |
Verified fixed -- Testathon 3/17/2023 🍀 |
Summary
When an aggregate element in the tree is navigated to, it provides a LAD Table view of its children. This is great when the children are leaf telemetry nodes, but when the children are themselves aggregates, the LAD view tries to displays "values" for those elements:
What's happening is that the first actual value within the nested aggregate is being lifted up as if it's the value of the aggregate, which is very misleading.
Proposed Solution
Provide the LAD Table view, but only show values for actual telemetry end points. For nested child aggregate elements, include them in the list, but don't show a timestamp or value. Consider adding a "Type" column to this view in particular (not all LAD Table views) that would clarify what type of object a given line item is.
At the very least, if we can't fix the view as suggested above, we should suppress the table view when it's not capable of displaying the right data.
Steps to Reproduce
Environment
Impact Check List
Additional Information
The text was updated successfully, but these errors were encountered: