-
Notifications
You must be signed in to change notification settings - Fork 29
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.
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
Small update to home load handling #512
Merged
Merged
Changes from all commits
Commits
Show all changes
105 commits
Select commit
Hold shift + click to select a range
ca47fe2
Initial commit for support of all schedule types, and fix for histori…
tmenguy 0962f81
Corrected devices to use hstoryMixin to get energy
tmenguy 7faa1ad
Merged historyMixin and Energy has they are effectively doing the sum…
tmenguy 4db8c5c
removed the daily update
tmenguy 9bfe32a
added a reset measures method to be used directly by external libreries
tmenguy aa841c4
reset now resets timings too
tmenguy f763785
Try to get a proper historical data in case of peak / off_peak energy…
tmenguy e8f3065
Fixed correctly the timing end/start of measures
tmenguy 628e610
Fixed correctly the timing end/start of measures
tmenguy 08ac5f6
Fixed correctly the timing end/start of measures
tmenguy cf198ec
support correct empty measures
tmenguy 3af2cd3
removed specidif energy exception to use th egeneric API error, 0lus …
tmenguy e413911
removed specidif energy exception to use th egeneric API error, 0lus …
tmenguy 4765304
removed specidif energy exception to use th egeneric API error, 0lus …
tmenguy 9efb0f3
removed specidif energy exception to use th egeneric API error, 0lus …
tmenguy fdce0cc
async update for all homes
tmenguy 2407b39
async update for all homes
tmenguy 9e82e1a
async update for all homes
tmenguy 938080a
async update for all homes
tmenguy 5ad1fe1
async update for all homes
tmenguy 3ab9ec0
async update for all homes
tmenguy 1cb8d3e
async update for all homes
tmenguy 2c3e4ea
async update for all homes
tmenguy 2fde124
async update for all homes
tmenguy 72508f3
added some tests for historical data
tmenguy e0d6a1f
added some tests for historical data
tmenguy 024344a
added a way to get the data sum
tmenguy 252280e
added throttling exception
tmenguy 9b4c539
fix for num calls estimation
tmenguy c88c2f3
fix for num calls estimation
tmenguy d593bfa
changed type for energy
tmenguy 645b3d7
changed type for energy
tmenguy 9c6e16c
Added historical retrival for other state, begining only for power (t…
tmenguy 936aeb9
Added historical retrival for other state, begining only for power (t…
tmenguy 7f626ed
Added historical retrival for other state, begining only for power (t…
tmenguy 8431020
Added historical retrival for other state, begining only for power (t…
tmenguy c3b55ef
Added historical retrival for other state, begining only for power (t…
tmenguy 001f92e
Added historical retrival for other state, begining only for power (t…
tmenguy 6170e9f
Added historical retrival for other state, begining only for power (t…
tmenguy 1ca53b7
Merge remote-tracking branch 'upstream/master' into development
tmenguy 8350914
Merge remote-tracking branch 'upstream/development' into development
tmenguy acc7331
Added historical retrival for other state, begining only for power (t…
tmenguy 59f6cc9
all tests pass
tmenguy 7b9b6dc
added energy sum
tmenguy 89c7309
added energy sum
tmenguy 238ccea
added energy sum
tmenguy abf5680
added energy sum
tmenguy 666297b
PEP cleaning to prepare PR
tmenguy 25d7bfa
adjusting estimation calculus
tmenguy 0f8731e
adjusting estimation calculus
tmenguy adca869
adjusting estimation calculus
tmenguy ce496cf
slight refactor for readability
tmenguy 4a9c9fb
Support error for unreachable home and bridges ....it is an error, no…
tmenguy 3e5c70e
Support error for unreachable home and bridges ....it is an error, no…
tmenguy ce7ca84
remove global energy sum support, no needed anymore, was adding compl…
tmenguy c26f66d
remove global energy sum support, no needed anymore, was adding compl…
tmenguy a756d1b
refactor to use self.homes where it should be
tmenguy d2dceec
setting schedules list back as the global list of schedules, for retr…
tmenguy 018c04c
Black....
tmenguy f161af9
Ruff....
tmenguy ca51a00
Ruff....
tmenguy 78dcd95
Ruff....
tmenguy c04947b
Ruff.... then black funny game
tmenguy 54718f6
black.... then ruff funny game
tmenguy 52bf849
black and ruff game
tmenguy 72796c6
black and ruff game
tmenguy 051005f
YABT .. yet another Black Test
tmenguy 4fe2987
Cleaned supported homes with a way simpler approach, thx @cgtobi
tmenguy 8d6a9bc
keep black API
tmenguy ec038ba
keep black API
tmenguy 415fcf7
Added back a list of available homes to be able to select the one nee…
tmenguy bae3959
white space
tmenguy 61a85d2
Merge remote-tracking branch 'refs/remotes/upstream/development' into…
tmenguy d4bb6e4
Merge remote-tracking branch 'refs/remotes/upstream/development' into…
tmenguy 55761eb
Support NLE "connected ecocometer" that is a bridge
tmenguy 33fc145
Support NLE "connected ecocometer" that is a bridge...and has no pow…
tmenguy 8ed5bac
Support NLE "connected ecocometer" that is a bridge...and has no pow…
tmenguy d544802
Support NLE "connected ecocometer" that is a bridge...and has no pow…
tmenguy c057ff2
Black / Rust
tmenguy 16af56a
Black / Rust
tmenguy 9409e6b
Black / Rust
tmenguy b476257
Remove schedule code API calls, etc ... to make it compatible with ol…
tmenguy 2bfdb12
Remove schedule code API calls, etc ... to make it compatible with ol…
tmenguy d791f75
Remove schedule code API calls, etc ... to make it compatible with ol…
tmenguy 6ec554e
Remove schedule code API calls, etc ... to make it compatible with ol…
tmenguy aa1240f
Black / Rust
tmenguy 9be36e2
Black / Rust
tmenguy ea5dd4e
Black / Rust
tmenguy c8bca7d
Black / Rust
tmenguy 4d204c9
Black / Rust
tmenguy d80c562
Black / Ruff
tmenguy ad7c58e
Black / Ruff
tmenguy 0673532
some first remarks from PR fixes
tmenguy ae6b4fc
removed home update boolean as output, and raise conditionally a reac…
tmenguy 06bc34c
removed home update boolean as output, and raise conditionally a reac…
tmenguy 542f9b9
rieman energy method extraction
tmenguy c608134
last PR comments
tmenguy ba590c6
black / ruff
tmenguy 79ce301
Merge branch 'jabesq-org:development' into development
tmenguy 5de319a
Merge branch 'jabesq-org:development' into development
tmenguy 6f5c8bb
Merge branch 'jabesq-org:development' into development
tmenguy 2da7edc
better error handling
tmenguy dbb3e4b
Merge remote-tracking branch 'origin/development' into development
tmenguy b89b07b
Merge branch 'jabesq-org:development' into development
tmenguy e42f543
add a comment
tmenguy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
suggestion (performance): Consider optimizing the
has_an_update
flag assignmentThe current implementation sets
has_an_update = True
for each person update. Consider moving this assignment before the loop for efficiency. Also, review if all person updates truly imply a house update, and ensure the flag is being used effectively in the broader context.