Skip to content

[Folders] Add LAD Table and Stacked Plot views#8213

Open
heathdutton wants to merge 1 commit intonasa:masterfrom
heathdutton:8105-default-telemetry-views-for-folders
Open

[Folders] Add LAD Table and Stacked Plot views#8213
heathdutton wants to merge 1 commit intonasa:masterfrom
heathdutton:8105-default-telemetry-views-for-folders

Conversation

@heathdutton
Copy link

Closes #8105

Describe your changes:

Adds default telemetry views for folders, allowing users to visualize telemetry items within a folder's composition as a LAD Table or Stacked Plot.

  • Add FolderLadTableViewProvider - enables LAD Table view for folders
  • Add FolderStackedPlotViewProvider - enables Stacked Plot view for folders
  • Add defensive filtering in LadTable.vue and StackedPlot.vue to gracefully skip non-telemetry items in composition

Testing:

  1. Create a folder containing telemetry objects (e.g., from the example generator)
  2. Navigate to the folder
  3. Use the view switcher to select "LAD Table" or "Stacked Plot" if you like
  4. Verify telemetry data displays as expected
  5. Verify folders containing non-telemetry items (e.g., other folders) don't cause errors

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Is this a notable change that will require a special callout in the release notes? For example, will this break compatibility with existing APIs or projects that consume these plugins?

Author Checklist

  • Changes address original issue?
  • Tests included and/or updated with changes?
  • Has this been smoke tested?
  • Have you associated this PR with a type: label? Note: this is not necessarily the same as the original issue.
  • Have you associated a milestone with this PR? Note: leave blank if unsure.
  • Testing instructions included in associated issue OR is this a dependency/testcase change?

Reviewer Checklist

  • Changes appear to address issue?
  • Reviewer has tested changes by following the provided instructions?
  • Changes appear not to be breaking changes?
  • Appropriate automated tests included?
  • Code style and in-line documentation are appropriate?

Copilot AI review requested due to automatic review settings December 12, 2025 05:24
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds default telemetry visualization capabilities to folders by introducing LAD Table and Stacked Plot view providers. The implementation enables users to view telemetry items within a folder's composition using these visualization types, with defensive filtering to handle non-telemetry items gracefully.

Key changes:

  • Adds two new view providers for folders: FolderLadTableViewProvider and FolderStackedPlotViewProvider
  • Implements defensive filtering in LadTable.vue and StackedPlot.vue to skip non-telemetry objects

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/plugins/folderView/FolderLadTableViewProvider.js New class-based view provider enabling LAD Table view for folder types
src/plugins/folderView/FolderStackedPlotViewProvider.js New function-based view provider enabling Stacked Plot view for folder types
src/plugins/folderView/plugin.js Registers the two new view providers with the OpenMCT instance
src/plugins/LADTable/components/LadTable.vue Adds telemetry check in addItem method to filter out non-telemetry objects
src/plugins/plot/stackedPlot/StackedPlot.vue Adds telemetry and overlay plot type check in addChild method to filter out non-plottable objects

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@heathdutton heathdutton force-pushed the 8105-default-telemetry-views-for-folders branch from ef57b0c to c912c3a Compare December 12, 2025 06:02
@heathdutton heathdutton reopened this Dec 12, 2025
Add default telemetry views for folders containing telemetry objects. When viewing a folder, users can now select LAD Table or Stacked Plot views to visualize telemetry items within the folder's composition. Simple test included.
@heathdutton heathdutton force-pushed the 8105-default-telemetry-views-for-folders branch from 1e4e3ac to f4b9e7b Compare February 15, 2026 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default telemetry views for folders

2 participants