-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[basicprofiles] Convert to relative unit in State Filter's Delta check #18127
Conversation
This pull request has been mentioned on openHAB Community. There might be relevant details there: https://community.openhab.org/t/state-filter-range-filter-profile/158025/58 |
470b991
to
3c248fa
Compare
I'll address that in another PR once the current PR backlog to StateFilter has been merged, just to avoid having to deal with merge conflicts. |
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.
Some suggestions. And also please look at #18144 please.
.../test/java/org/openhab/transform/basicprofiles/internal/profiles/StateFilterProfileTest.java
Outdated
Show resolved
Hide resolved
.../test/java/org/openhab/transform/basicprofiles/internal/profiles/StateFilterProfileTest.java
Outdated
Show resolved
Hide resolved
.../src/main/java/org/openhab/transform/basicprofiles/internal/profiles/StateFilterProfile.java
Show resolved
Hide resolved
3c248fa
to
458caeb
Compare
rebased |
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.
LGTM
Signed-off-by: Jimmy Tanagra <[email protected]>
458caeb
to
ea3dc22
Compare
Signed-off-by: Jimmy Tanagra <[email protected]>
ea3dc22
to
db1e876
Compare
rebased to resolve merge conflicts |
Is this classed as "enhancement" or "bug"? Ideally this should be backported so people can use it in 4.3 |
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.
Thanks, LGTM
#18127) * [basicprofiles] Convert to relative unit in State Filter's Delta check Signed-off-by: Jimmy Tanagra <[email protected]>
Reported in https://community.openhab.org/t/state-filter-range-filter-profile/158025/51