Skip to content

Add opt-in inactivity expiry for ui-chart series/topics#2094

Open
omidteimoori wants to merge 1 commit intoFlowFuse:mainfrom
omidteimoori:feat/ui-chart-inactivity-expiry
Open

Add opt-in inactivity expiry for ui-chart series/topics#2094
omidteimoori wants to merge 1 commit intoFlowFuse:mainfrom
omidteimoori:feat/ui-chart-inactivity-expiry

Conversation

@omidteimoori
Copy link
Copy Markdown

@omidteimoori omidteimoori commented Apr 10, 2026

Summary

  • add an opt-in ui-chart setting to expire inactive series/topics after a configurable timeout
  • remove only the inactive series from chart state so active series remain untouched
  • preserve existing ui-chart behavior when the option is disabled

Refs issue #2090

Add opt-in inactivity cleanup for ui-chart series/topics with a configurable timeout.\n\nRefs FlowFuse#2090
@colinl
Copy link
Copy Markdown
Collaborator

colinl commented Apr 10, 2026

Have you considered how this will work if the data points are added as a complete set of historical data from, for example, a database? Will all the legends suddenly disappear after the specified time?

@omidteimoori
Copy link
Copy Markdown
Author

Yes. In the current implementation, inactivity is measured from the last time a series was received by the dashboard, not from the datapoint’s own x-value. So if a chart is populated from a one-off historical dataset and no further updates arrive, those series will expire after the configured timeout. That is intentional for the live-data use case this option targets.
For historical/static datasets, users can simply leave the option disabled, in which case ui-chart behaves exactly as it does today.

@colinl
Copy link
Copy Markdown
Collaborator

colinl commented Apr 10, 2026

Would it not have been much simpler to limit it to timestamp x axes and use the timestamp of the latest sample?

The PR you have submitted is remarkably complex and will take a lot of time to review, I suspect you will find it difficult to find a developer to do that.

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.

2 participants