Skip to content

dashboard: prevent calibration settings loss on restart - #853

Closed
babo989 wants to merge 45 commits into
mainfrom
fix/calibration-settings-persistence
Closed

babo989 wants to merge 45 commits into
mainfrom
fix/calibration-settings-persistence

Conversation

@babo989

@babo989 babo989 commented Feb 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Reduce Node-RED context storage flush interval from 30s to 5s to prevent calibration data loss on restart.

@sonnyp sonnyp changed the title fix: prevent calibration settings loss on restart dashboard: prevent calibration settings loss on restart Feb 3, 2026
@sonnyp
sonnyp changed the base branch from update-dashboard to main February 3, 2026 17:16

@sonnyp sonnyp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

When you say "on restart" do you mean on restarting node-red or restarting the planktoscope?

Comment thread node-red/settings.cjs
module:"localfilesystem"
module: "localfilesystem",
config: {
// FIX: Flush every 5 seconds instead of default 30 to prevent

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
// FIX: Flush every 5 seconds instead of default 30 to prevent
// Flush every 5 seconds instead of default 30 to prevent

Comment on lines -2186 to +2187
"x": 650,
"y": 180,
"x": 710,
"y": 140,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

please revert all unrelated changes (flow.json)

@sonnyp

sonnyp commented Feb 16, 2026

Copy link
Copy Markdown
Collaborator

We think it's not an issue anymore.

@sonnyp sonnyp closed this Feb 16, 2026
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.

3 participants