Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

LAD table not quite as LAD as telemetry table #5111

Closed
3 of 5 tasks
davetsay opened this issue Apr 25, 2022 · 7 comments · Fixed by #5127
Closed
3 of 5 tasks

LAD table not quite as LAD as telemetry table #5111

davetsay opened this issue Apr 25, 2022 · 7 comments · Fixed by #5127

Comments

@davetsay
Copy link
Contributor

davetsay commented Apr 25, 2022

Summary

When a telemetry endpoint is added to both a LAD table and telemetry table, the LAD table does not display the most recent data whereas the telemetry table does.

Expected vs Current Behavior

LAD table and telemetry table with the same endpoint added should display the same latest data. LAD table does not display the latest data.

Steps to Reproduce

  1. Create a LAD table and add a telemetry endpoint
  2. Create a telemetry table and add the same telemetry endpoint as was added in step 1
  3. Add both tables to a display layout
  4. Set time conductor to realtime mode, and set end offset to 0
  5. Set time conductor to fixed time mode
  6. Compare the timestamps for the LAD table and the latest timestamp in the telemetry table

Note that manually setting the endtime for fixed time mode will give consistent values for the latest available data in that range.

Environment

  • Open MCT Version: 378a4ca
  • Deployment Type: VIPER ce8ba4f02230848be58ec592f55ba40bc4d62c7d
  • OS:
  • Browser: chrome

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?

Additional Information

@jvigliotta
Copy link
Contributor

jvigliotta commented Apr 27, 2022

Testing

  • what dave said in "Steps to Reproduce", minus where he messed up the formatting and linked to our very first merged PR
  • also, play around with LAD tables and time systems and modes (local vs fixed)
  • also LAD Table Sets...

@charlesh88
Copy link
Contributor

Verified fixed Testathon 4/28/22

@akhenry
Copy link
Contributor

akhenry commented Apr 28, 2022

Verified fixed

@jvigliotta
Copy link
Contributor

Testing

  • SAME as before... but, there is a yamcs bug
  • if the start time === datum timestamp, it will not show up. this is a YAMCS bug, and will not prevent this fix from passing

@akhenry
Copy link
Contributor

akhenry commented May 2, 2022

Verified fixed (aside from Yamcs issue)

@nikhilmandlik
Copy link
Contributor

Verified Fixed. Yamcs Issue not fixed

@davetsay
Copy link
Contributor Author

davetsay commented May 2, 2022

verified (locally since yamcs has a bug)

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

Successfully merging a pull request may close this issue.

6 participants