Skip to content
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

Merged
merged 2 commits into from
Feb 2, 2025

Conversation

jimtng
Copy link
Contributor

@jimtng jimtng commented Jan 19, 2025

@jimtng jimtng requested a review from J-N-K as a code owner January 19, 2025 00:35
@openhab-bot
Copy link
Collaborator

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

@jimtng jimtng force-pushed the statefilter-delta-relativeunit branch from 470b991 to 3c248fa Compare January 19, 2025 01:30
@jimtng jimtng changed the title [basicprofiles] Fix delta comparison for relative units in State Filter [basicprofiles] Convert to relative unit in State Filter's Delta check Jan 19, 2025
@andrewfg
Copy link
Contributor

@jimtng please have a look at #18122

@jimtng
Copy link
Contributor Author

jimtng commented Jan 20, 2025

@jimtng please have a look at #18122

#18122 (comment)

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.

Copy link
Contributor

@andrewfg andrewfg left a 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.

@andrewfg andrewfg added the enhancement An enhancement or new feature for an existing add-on label Jan 23, 2025
@jimtng jimtng force-pushed the statefilter-delta-relativeunit branch from 3c248fa to 458caeb Compare January 24, 2025 01:02
@jimtng
Copy link
Contributor Author

jimtng commented Jan 24, 2025

rebased

Copy link
Contributor

@andrewfg andrewfg left a comment

Choose a reason for hiding this comment

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

LGTM

@jimtng jimtng force-pushed the statefilter-delta-relativeunit branch from 458caeb to ea3dc22 Compare January 24, 2025 13:52
@jimtng jimtng force-pushed the statefilter-delta-relativeunit branch from ea3dc22 to db1e876 Compare January 24, 2025 13:54
@jimtng
Copy link
Contributor Author

jimtng commented Jan 24, 2025

rebased to resolve merge conflicts

@jimtng
Copy link
Contributor Author

jimtng commented Jan 27, 2025

Is this classed as "enhancement" or "bug"? Ideally this should be backported so people can use it in 4.3

@andrewfg andrewfg added bug An unexpected problem or unintended behavior of an add-on and removed enhancement An enhancement or new feature for an existing add-on labels Jan 27, 2025
@andrewfg andrewfg requested a review from a team February 1, 2025 18:40
Copy link
Contributor

@lsiepel lsiepel left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@lsiepel lsiepel merged commit 019fb8a into openhab:main Feb 2, 2025
2 checks passed
lsiepel pushed a commit that referenced this pull request Feb 2, 2025
#18127)

* [basicprofiles] Convert to relative unit in State Filter's Delta check

Signed-off-by: Jimmy Tanagra <[email protected]>
@lsiepel lsiepel added the patch A PR that has been cherry-picked to a patch release branch label Feb 2, 2025
@lsiepel lsiepel added this to the 5.0 milestone Feb 2, 2025
@jimtng jimtng deleted the statefilter-delta-relativeunit branch February 3, 2025 05:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants