Adds new check to clearData that enables data clearing on overlay plots. #7961
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes
Describe your changes:
Closes #7947
This change adds a new case to the existing 3 logic scenarios in the ClearData function (new case in bold):
I maintain that the 3rd case, compositionPathContainsId may have originally been intended to capture the 4th case, but I don't want to break someone else's work so it is safer to just cover the currently missing 4th case.
To test this change, you'll have to do the following:
to the TelemetryMeanProvider.
Note: This is much easier to test with a server responding to telemetry requests as opposed to using sine wave generators.
All Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist