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

[sbus] Initial contribution #18019

Open
wants to merge 1,254 commits into
base: main
Choose a base branch
from
Open

Conversation

cipianpascu
Copy link

No description provided.

@cipianpascu cipianpascu requested review from GiviMAD and a team as code owners January 2, 2025 07:46
ngreatorex and others added 28 commits January 2, 2025 09:49
…b#17333)

At least the Deebot 900 sends an unusual error response in case of
expired token.

Fixes openhab#15961

Signed-off-by: Danny Baumann <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…x on on upgrade (openhab#17703)

* [jsscripting] Upgrade to openhab-js 5.8.0

Signed-off-by: Florian Hotze <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* [ism8] Add channel types for heat pumps

Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…hab#17661)

* Events were dispatched but group id was missing

Signed-off-by: Gaël L'hopital <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…t temperature (openhab#17662)

* setpointDefaultDuration was not always considered

Signed-off-by: Gaël L'hopital <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…t filter (openhab#16105)

* Extract time-based event search strategies into a separate class

This allows extensions without having to adapt the logic in
BiweeklyPresentableCalendar.

Signed-off-by: Christian Heinemann <[email protected]>
Co-authored-by: Leo Siepel <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* Provide LED channel for access

Signed-off-by: Jacob Laursen <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Nick Waterton <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* Network

Signed-off-by: Leo Siepel <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* Yet another website underlaying API modification
* Correction for current and previous week, month, year
* Added unitHing
* Switch peek power to kVA
* Adding new cookie and user agent

Signed-off-by: Gaël L'hopital <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Artur-Fedjukevits <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* First version

Signed-off-by: Jan Gustafsson <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* stabilize nanoleaf binding:suppress ipv6 addresses in controller discovery

Signed-off-by: Stefan Höhn <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Konstantin Polihronov <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…ico (openhab#17723)

* Added discovery of DivaSmartDimmer and PaddleSwitchPico.  Also, it will not print out ID of any unrecongnized device in the logs.

Signed-off-by: Jeff James <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* Show encrypted frames without a matching key using console command
  "openhab:knx list-unknown-ga"; sort output numerically by GA.
* Add trace logging to show decryption error due to missing key.
  Previously, those frames were silently dropped unless logging for
  Calimero was explicitly enabled as well.

Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* [shelly] Add support for Shelly Plus RGBW PM

Signed-off-by: Jan N. Klug <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Jacob Laursen <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* [avmfritz] Fix incorrect state of dimmable bulb

Fixes openhab#17609

Signed-off-by: Fabian Girgert <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…ttp (openhab#17570)

Bumps org.eclipse.jetty:jetty-server from 9.4.54.v20240208 to 9.4.55.v20240627.

---
updated-dependencies:
- dependency-name: org.eclipse.jetty:jetty-server
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Ciprian Pascu <[email protected]>
Related to openhab#17636

Signed-off-by: Laurent Garnier <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
I made various changes and fixes to the ONVIF connection in the IpCamera binding.

Signed-off-by: David Masshardt <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* [solarman] Added LSE-3 (LAN Stick Logger) Support (openhab#17559)

Signed-off-by: Peter Kretz <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
andrewfg and others added 11 commits January 2, 2025 09:49
…penhab#17919)

* [various] process color temperature quantity type commands

Signed-off-by: Andrew Fiddian-Green <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
…nhab#17680)

* [miio] add support for Smart Air Purifier 4 Lite zhimi.airp.rmb1

Signed-off-by: Marcel Verpaalen <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
* [jsscripting] Upgrade GraalJS to 23.0.6

Signed-off-by: Florian Hotze <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
Signed-off-by: Ciprian Pascu <[email protected]>
@jlaur jlaur added the new binding If someone has started to work on a binding. For a new binding PR. label Jan 2, 2025
@jlaur jlaur changed the title Feature/s bus [sbus] Initial contribution Jan 2, 2025
@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/new-transport-protocol-s-bus-g4/152769/1

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.

Thansk for this updated PR and the contribution. Many of my comments are the same issue on defferent locations. So i think with 1 or 2 more iterations we can finish this PR.

Besides these comments, also be sure all compile warnings and SAT errors are fixed. Also don;t forget to add yourself to the CODEOWNERS file.

bundles/org.openhab.binding.sbus/DEVELOPERS.md Outdated Show resolved Hide resolved
bundles/org.openhab.binding.sbus/README.md Show resolved Hide resolved
bundles/org.openhab.binding.sbus/README.md Outdated Show resolved Hide resolved
bundles/org.openhab.binding.sbus/README.md Outdated Show resolved Hide resolved
bundles/org.openhab.binding.sbus/README.md Outdated Show resolved Hide resolved

// Iterate over all channels and update their states with corresponding temperatures
for (Channel channel : getThing().getChannels()) {
SbusChannelConfig channelConfig = channel.getConfiguration().as(SbusChannelConfig.class);
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a check isLinked so only channels are iterated that are actually linked to an item. Saves some cpu-cycles for existing but not used channels.

Copy link
Author

Choose a reason for hiding this comment

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

I don't know how to do it. But the call to sbus devices is done only once for all the channels. It doesn't have much of an impact.

@cipianpascu
Copy link
Author

hey @lsiepel, please help me with the spotless build issue. I don't understand why is it passing on my local computer but fails on github. Thank you

@lsiepel
Copy link
Contributor

lsiepel commented Jan 11, 2025

hey @lsiepel, please help me with the spotless build issue. I don't understand why is it passing on my local computer but fails on github. Thank you

Your branch is 18 commits behind, I think some are spotless related (version upgrade). Please update your main and afterwards your feature branch.

@cipianpascu
Copy link
Author

hey @lsiepel, please help me with the spotless build issue. I don't understand why is it passing on my local computer but fails on github. Thank you

Your branch is 18 commits behind, I think some are spotless related (version upgrade). Please update your main and afterwards your feature branch.

After the update, I had the same problem on my local computer. Apparently, there is an issue with the Eclipse JDT formatter version and Lombok dependencies, project wide.

Signed-off-by: Ciprian Pascu <[email protected]>
@GiviMAD GiviMAD removed their request for review January 12, 2025 16:46
@lsiepel
Copy link
Contributor

lsiepel commented Jan 12, 2025

hey @lsiepel, please help me with the spotless build issue. I don't understand why is it passing on my local computer but fails on github. Thank you

Your branch is 18 commits behind, I think some are spotless related (version upgrade). Please update your main and afterwards your feature branch.

After the update, I had the same problem on my local computer. Apparently, there is an issue with the Eclipse JDT formatter version and Lombok dependencies, project wide.

It’s not spotless it is SAT. Please check the errors to fix them

Ciprian Pascu added 2 commits January 12, 2025 19:34
Signed-off-by: Ciprian Pascu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new binding If someone has started to work on a binding. For a new binding PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.