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

Correlation Telemetry Object throws errors and does not emit any correlated values #5539

Closed
2 of 7 tasks
akhenry opened this issue Jul 19, 2022 · 2 comments
Closed
2 of 7 tasks

Comments

@akhenry
Copy link
Contributor

akhenry commented Jul 19, 2022

Summary

  • Creating a correlation telemetry point does not result in any correlated data being emitted
  • Errors are shown in the JavaScript console
  • The errors persist even after navigating away from the correlation telemetry object, suggesting the existence of a memory leak.

Steps to Reproduce

  1. Create two sine wave generators
  2. Create a new correlation telemetry point and select the previously created sine wave generators as the telemetry sources
  3. Save the new correlation telemetry point
  4. Observe that no data is displayed
  5. Open the JavaScript console and observe errors similar to below:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'namespace')
    at Object.makeKeyString (object-utils.js?40a1:98:1)
    at ObjectAPI.makeKeyString (ObjectAPI.js?99d1:595:16)
    at ObjectAPI.get (ObjectAPI.js?99d1:193:1)
    at getTelemetryObject (plugin.js?6ac3:7:1)
    at Object.request (plugin.js?6ac3:80:1)
    at TelemetryAPI.request (TelemetryAPI.js?7b82:281:1)

Environment

  • Open MCT Version: 2bfe632 (release/2.0.5)
  • Deployment Type: VIPER test

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

@jvigliotta
Copy link
Contributor

This was made in another repo, closing as it's been fixed.

@jvigliotta
Copy link
Contributor

Verified Fixed: Testathon 8/2/22

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

No branches or pull requests

3 participants