Skip to content

Commit

Permalink
fix output and add accumulation label
Browse files Browse the repository at this point in the history
  • Loading branch information
scottbell committed Oct 10, 2024
1 parent 4de0354 commit 3b78d4d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/plugins/comps/CompsManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export default class CompsManager extends EventEmitter {
testValue: 0,
timeMetaData,
accumulateValues: false,
sampleSize: null
sampleSize: 10
});
this.emit('parameterAdded', this.#domainObject);
}
Expand Down
6 changes: 4 additions & 2 deletions src/plugins/comps/components/CompsView.vue
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,9 @@
aria-label="Toggle Parameter Accumulation"
></span>
</label>
<div v-if="!isEditing && parameter.accumulateValues">
- accumulating values with sample size {{ parameter.sampleSize }}
</div>

<span v-if="isEditing && parameter.accumulateValues" class="c-test-datum__string"
>Sample Size</span
Expand Down Expand Up @@ -207,8 +210,7 @@ const props = defineProps({

onBeforeMount(async () => {
const telemetryOptions = {

Check warning on line 212 in src/plugins/comps/components/CompsView.vue

View check run for this annotation

Codecov / codecov/patch

src/plugins/comps/components/CompsView.vue#L211-L212

Added lines #L211 - L212 were not covered by tests
size: 1,
strategy: 'latest'
strategy: 'minmax'
};
outputTelemetryCollection = openmct.telemetry.requestCollection(domainObject, telemetryOptions);
outputTelemetryCollection.on('add', telemetryProcessor);
Expand Down

0 comments on commit 3b78d4d

Please sign in to comment.