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

[Plot] Unable to annotate the plot view of a Telemetry Point #6209

Closed
2 of 7 tasks
akhenry opened this issue Jan 26, 2023 · 2 comments · Fixed by #6199 or #6274
Closed
2 of 7 tasks

[Plot] Unable to annotate the plot view of a Telemetry Point #6209

akhenry opened this issue Jan 26, 2023 · 2 comments · Fixed by #6199 or #6274

Comments

@akhenry
Copy link
Contributor

akhenry commented Jan 26, 2023

Summary

When a user navigates directly to the plot view of a telemetry point (this is the default view of a telemetry point) and attempts to annotate it, the annotation is not created, and an error appears in the JavaScript console

Expected vs Current Behavior

The annotation should be created and shown in the plot. Although Telemetry Points are immutable annotating them is expressly supported by our data architecture.

The issues is almost certainly occurring when we attempt to update the targetDomainObjectAnnotated timestamp on the Telemetry Object itself. We need some guard code to not attempt this if the object is immutable.

Steps to Reproduce

  1. Navigate to a an immutable Telemetry Object (SWG won't work for this purpose, they are mutable)
  2. You should see a telemetry plot.
  3. Pause the plot and select some points to annotate.
  4. In the inspector, click on the Annotations tab
  5. Select a tag to add to the plot
  6. Note that the selected tag does not get updated
  7. Open the JavaScript console and observe the following error:
Object provider does not support saving

Should be reproducible with Open MCT-Yamcs

Environment

  • Open MCT Version: 2.1.6
  • 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

@akhenry akhenry changed the title [Plot] Unable to the plot view of a Telemetry Point [Plot] Unable to annotate the plot view of a Telemetry Point Jan 26, 2023
@scottbell scottbell mentioned this issue Jan 27, 2023
15 tasks
@scottbell scottbell linked a pull request Jan 27, 2023 that will close this issue
15 tasks
@shefalijoshi shefalijoshi added this to the Target:2.1.6 milestone Feb 1, 2023
@ozyx
Copy link
Member

ozyx commented Feb 2, 2023

Not fixed -- Testathon 02-02-2023

@ozyx ozyx reopened this Feb 2, 2023
@scottbell
Copy link
Contributor

scottbell commented Feb 3, 2023

Not fixed -- Testathon 02-02-2023

This is a different issue, but manifesting as the same problem. I'm assigning the namespace in AnnotationsAPI with the same value as the thing it's annotating. If using YAMCS, this is going to be taxonomy, which when saving, is going to throw an exception as the YAMCS object provider doesn't support saving. I think we've got at least three options:

  1. Walk through the providers in the AnnotationsAPI and find the first provider that supports saving. Use that provider's keyspace for newly created annotations.
  2. Use the MyItems namespace.
  3. Use `` as the namespace (which is what MyItems uses as a default).
  4. Have a plugin configuration that tells OpenMCT which provider to use for persisting annotations, e.g. in the Tags plugin.

I've implemented option 4, but please let let me know what you think @akhenry

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