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

Openknx merge #289

Merged
merged 59 commits into from
Aug 9, 2024
Merged

Openknx merge #289

merged 59 commits into from
Aug 9, 2024

Conversation

thelsing
Copy link
Owner

@thelsing thelsing commented Aug 9, 2024

No description provided.

Waldemar Porscha and others added 30 commits January 16, 2024 00:39
- old implementation is still there for compatibility reasons
- new implementation reflects the case "uninitialized while transmitting"
- Just states "Updated", "WriteRequest" and "Ok" remove uninitialized state
…nversion by `valueNoSendCompare(..)`

* Return if Value was Changed
* Always Set the First Value
* Copy on Changes Only
* Make Comparison Independent of Sending
* Extend Doc of `valueNoSendCompare(..)`; Add Note for using `valueNoSend(..)`
custom_hwids.py was never executed in the ci pipeline, but the missing script was just a warning. With Plattform I/O Core 6.1.11 missing scripts are errors, so the pipeline fails.
# Conflicts:
#	src/knx/group_object.cpp
reenable rxProcess in loop
fixes a memory leak
adds a tx queue size limit with warning.
use newest atmelsam plattform
Add function to group object to send a value only if it was changed.
traxanos and others added 28 commits June 3, 2024 10:38
…ction is defined

otherwise it is useless anyway and causes warnings.
* String \0 terminated in group objects

* Remove copy constructor, fix bugs in setting buffer to 0

* Remove copy constructor in GroupObject
Support 0x091A IP-Router better, including KNX IP Tunneling and a lot of smaller fixes
@thelsing thelsing merged commit 44f90bb into master Aug 9, 2024
8 of 9 checks passed
@thelsing thelsing deleted the openknx-merge branch August 9, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants