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

Split topic values into individual telemetry points #31

Open
AWatk opened this issue Dec 6, 2017 · 2 comments · May be fixed by #37
Open

Split topic values into individual telemetry points #31

AWatk opened this issue Dec 6, 2017 · 2 comments · May be fixed by #37
Assignees
Milestone

Comments

@AWatk
Copy link
Member

AWatk commented Dec 6, 2017

combine with #27 for properly referencing a topic value and then change object provider to return an individual telemetry object for each value. Topics will then be folders containing these telemetry points.

@AWatk AWatk self-assigned this Dec 6, 2017
@AWatk AWatk added this to the rosmct 0.2 milestone Dec 6, 2017
@AWatk
Copy link
Member Author

AWatk commented Jan 11, 2018

Splitting each topic field into its own telemetry point means that topics that contain other topics as a field will have nested folders.

This is necessary for visualization (I think) because there is a difference between a telemetry datum object (which needs to contain only a single value) and an openMCT form that can contain any number of telemetry datums that can be displayed on the same graph.

see the sine generator example for the use of such a form.
the generator can be installed via `openmct.install(openmct.plugins.Generator())
plugin code is here https://github.com/nasa/openmct/tree/a51b9bc63f738e04cb877626371b07bd6b924a27/example/generator

@AWatk
Copy link
Member Author

AWatk commented Jan 11, 2018

This complicates #24

Perhaps the form to visualize a topic should be auto-generated as well? Then the publishing interface can just be on that form instead of tied to a telemetry point.

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.

2 participants