Skip to content

Increase various ranges and resolutions#78

Merged
bvweerd merged 5 commits into
bvweerd:devfrom
anssih:increase-ranges
Aug 13, 2025
Merged

Increase various ranges and resolutions#78
bvweerd merged 5 commits into
bvweerd:devfrom
anssih:increase-ranges

Conversation

@anssih
Copy link
Copy Markdown
Contributor

@anssih anssih commented Aug 13, 2025

📝 What’s Changed?

  • Increase resolution of diagnostics sensors
  • Increase maximum sample time to 600 seconds
  • Increase range of coefficients

🔍 Why is this Change Needed?

  • Useful for slow-and-little-moving process variables

🧪 How Was This Tested?

  • Added or updated unit tests
  • Manually tested in development environment
  • CI/CD pipeline passed successfully

✅ Checklist

  • Code follows the style guide
  • All tests are passing
  • Documentation updated if needed
  • No breaking changes (or clearly documented)

📸 Screenshots / Logs (Optional)

(e.g., Home Assistant UI, terminal output, etc.)

📎 Additional Notes

  • ...

anssih and others added 5 commits August 13, 2025 16:27
E.g. on systems with very slowly moving process variable a large Kd and
small Ki may be needed.

Increase the range at both ends to accommodate such cases.
On systems with slowly-changing process variable a longer sample time
may be warranted.

Increase the maximum allowed value from 60 seconds to 600 seconds.
Increase resolution of diagnostics sensors to improve their usability in
systems where the process variable movements are small and slow.
@bvweerd bvweerd merged commit 6349a04 into bvweerd:dev Aug 13, 2025
3 checks passed
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