Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Precision Forge #82

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 215 commits into from
Jan 18, 2025
Merged

Precision Forge #82

merged 215 commits into from
Jan 18, 2025

Conversation

@Alexwijn
Copy link
Owner

@Alexwijn Alexwijn commented Jan 11, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

Based on the comprehensive summary, here are the updated release notes:

  • New Features

    • Added manufacturer selection during configuration
    • Introduced boiler status and manufacturer sensors
    • Enhanced error handling for device connections
    • Improved heating system state tracking
    • Added new functionality for setpoint adjustments

  • Improvements

    • Refined temperature and setpoint management
    • Added more granular device status reporting
    • Enhanced type annotations for better code clarity
    • Improved time tracking and calculation methods
    • Streamlined manufacturer resolution process

  • Bug Fixes

    • Updated serial device connection handling
    • Fixed potential issues with heating loop control

  • Refactoring

    • Restructured manufacturer and device management
    • Simplified configuration options
    • Removed deprecated minimum setpoint adjustment factor
    • Enhanced internal state management for simulators and coordinators

@Alexwijn
Dropped some obsolete variables and improved the dynamic minimum setp…
38bc33e 
…oint setup
@Alexwijn
Send MI=500 if it's an Ideal or Immergas boiler
f96ff01
@Alexwijn
Typo?
704428d
@Alexwijn
Make the adjustment factor more aggressive
b9dbb9c
@Alexwijn
Add support to have seperated modes for minimum setpoint
cc8044b
@Alexwijn
Making sure the boiler temperature isn't none
bdbb2b3
@Alexwijn
Use more dynamic adjustments to the minimum setpoint
d45361c
@Alexwijn
Change when we "warming up"
2e449ba
@Alexwijn
Cleanup
0f0ac2e
@Alexwijn
Change the order of execution
c53fa3f
@Alexwijn
Cleaning up
5e01d68
@Alexwijn
Make sure we only calculate when requesting heat
5233d61
@Alexwijn
Improve the minimum setpoint more
5b84bc2
@Alexwijn
Make sure we round the minimum setpoint
cafcf53
@Alexwijn
Some sanity checks
cceab92
@Alexwijn
Improved the logic to turn off "warming up"
1c629c6
@Alexwijn
Typo?
7677600
@Alexwijn
Some sanity checks
e4310fe
@Alexwijn
Some more sanity checks
36cc182
@Alexwijn
Wait a bit longer for adjustments
0614094
@Alexwijn
Do not use filtered temperatures since (should) change fast
373608d
@Alexwijn
Reduce the "change"
e3ffa51
@Alexwijn
Change the order (again)
8570be2
@Alexwijn
Added some sanity checks
433a4a2
@Alexwijn
Improved the PWM indicator without the need for OPV
67f16c7
@Alexwijn
Improved indicator of when going to "warming up"
f6e0a94
@Alexwijn
Applied "boiler_temperature_change" fix
888b102
@Alexwijn
Add some error handling for Sentry
3531c5f
@Alexwijn
Merge branch 'mqtt-coordinators' into improved-minimum-setpoint
a30979b
@Alexwijn
Merge branch 'develop' into improved-minimum-setpoint
db79568
Alexwijn and others added 24 commits January 13, 2025 18:59
@Alexwijn
Cleanup
44594b3
@Alexwijn
Add support for forcing a certain setpoint, so we don't do +10 on HA …
d16bec8 
…restart
@Alexwijn
Make sure we wait for the tracking before "overshooting"
74c597a
@Alexwijn
Re-added "Exceeds setpoint significantly" when tracking
b158a4e
@Alexwijn
Add support for "OVERSHOOT_STABILIZED"
fb123cb
@Alexwijn
Relax the AT_SETPOINT status
e58ad90
@Alexwijn
Optimized Boiler Cold Temperature
41b88ff
@Alexwijn
Clean up the boiler status
5127499
@Alexwijn
Typo?
aaba3a0
@Alexwijn
Add HEATING_UP when we are below 2 degrees BT
6e53f79
@Alexwijn
Removed "NEAR_SETPOINT"
0a15b3c
@Alexwijn
Re-added "NEAR_SETPOINT"
58b9b32
@Alexwijn
Drop obsolete method
85e0289
@Alexwijn
Make we delay the adjuster with 6 seconds
b9d259b
@Alexwijn
Make sure we don't disable PWM too early
5113899
@Alexwijn
Cleanup
1590d33
@Alexwijn
Add support for a connected thermostat which may take control of hvac…
7a18545 
… mode and the target temperature
@Alexwijn
Bump version
7b930e1
@Alexwijn
Cleaning and moving some code
37ff9aa
@Alexwijn
Merge branch 'develop' into feature/connected-thermostat
bef1488
@Alexwijn
Renamed the thermostat sync checkbox
1208633
@Alexwijn
Typo?
fdf09de
@Alexwijn
Merge pull request #83 from Alexwijn/feature/connected-thermostat
661c657 
Connected Thermostat
@Alexwijn
Add support for "maximum_setpoint_value"
a74d44d
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 18, 2025

✅ Actions performed

Summary regeneration triggered.

@Alexwijn Alexwijn changed the title New release Precision Forge Jan 18, 2025
@Alexwijn Alexwijn merged commit 1a11dab into master Jan 18, 2025
7 checks passed
@sentry-io
Copy link

sentry-io bot commented Jan 18, 2025

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ TypeError: '<' not supported between instances of 'NoneType' and 'int' custom_components.sat.coordinator in device_status View Issue

Did you find this useful? React with a 👍 or 👎

@coderabbitai coderabbitai bot mentioned this pull request Jan 25, 2025
Merged
@coderabbitai coderabbitai bot mentioned this pull request Feb 16, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Alexwijn @Thijssvd