Skip to content
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

Closed
2 of 7 tasks
charlesh88 opened this issue Nov 1, 2022 · 6 comments · Fixed by #6406
Closed
2 of 7 tasks

Aggregate LAD Table view doesn't work properly when children are aggregates #5939

charlesh88 opened this issue Nov 1, 2022 · 6 comments · Fixed by #6406

Comments

@charlesh88
Copy link
Contributor

charlesh88 commented Nov 1, 2022

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:

Screen Shot 2022-10-31 at 10 54 52 AM

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

  1. View an aggregate node that has an aggregate as a child.
  2. Observe that the LAD Table view erroneously displays a value for the child aggregate.

Environment

  • Open MCT Version: 2.1.2
  • Deployment Type: any
  • OS: all
  • Browser: any

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@akhenry
Copy link
Contributor

akhenry commented Feb 16, 2023

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)

@scottbell scottbell self-assigned this Mar 2, 2023
@scottbell
Copy link
Contributor

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.

@charlesh88
Copy link
Contributor Author

This issue is related to akhenry/openmct-yamcs#281.

@scottbell
Copy link
Contributor

@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"?

@scottbell
Copy link
Contributor

@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
Here's what I've got so far:
image

@ozyx
Copy link
Contributor

ozyx commented Mar 17, 2023

Verified fixed -- Testathon 3/17/2023 🍀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants