Skip to content

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Dec 25, 2025

See zip-file download at https://www.alpha-innotec.com/de/produkte/zubehor/smart-home---apps/knx-schnittstelle

Note: The min/max values now relates to the raw value

@github-actions
Copy link

github-actions bot commented Dec 25, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __init__.py251996%52, 55–59, 273–274, 279
   __main__.py23230%3–52
   calculations.py24675%327, 335, 340–343
   datatypes.py387199%121
   discover.py58198%62
   parameters.py24292%1212, 1223
   visibilities.py12192%403
luxtronik/scripts
   __init__.py16160%3–28
   discover_shi.py60600%8–98
   dump_changes.py43430%7–85
   dump_luxtronik.py26260%6–52
   dump_shi.py19190%7–40
   performance_shi.py63630%7–89
   watch_shi.py38380%7–85
luxtronik/shi
   contiguous.py135299%261–262
TOTAL218631086% 

Tests Skipped Failures Errors Time
324 0 💤 0 ❌ 0 🔥 9.045s ⏱️

@kbabioch
Copy link
Collaborator

Great find, finally we have a official documentation of this interface. At least as official as it will get.

@Guzz-T
Copy link
Contributor Author

Guzz-T commented Dec 26, 2025

I just noticed that there is more to do here. I'll get back to you as soon as I'm done.

@kbabioch kbabioch changed the title Update smart home interface definitions according to (in)official documentation WIP: Update smart home interface definitions according to (in)official documentation Dec 27, 2025
@kbabioch
Copy link
Collaborator

I've put it into WIP, so it won't be merged by accident ;-).

@Guzz-T Guzz-T changed the title WIP: Update smart home interface definitions according to (in)official documentation Update smart home interface according to (in)official documentation Dec 27, 2025
@Guzz-T
Copy link
Contributor Author

Guzz-T commented Dec 27, 2025

The pull request now contains a large number of changes. However, all of them have been added based on new findings from the documentation. Would you prefer to split these changes across several pull requests?

…r string value

Mainly to treat "x" and "Unknown_x" as equivalent.
…rs correctly

For example, 4-byte integer values are stored in two 2-byte registers with bit index 31..16 in the first register and 15..0 in the second register.
Due to the register size, the smart home interface often uses 16-bit signed values (for temperatures, ...).
This data type can handle values where each bit has a specific meaning.
@Guzz-T
Copy link
Contributor Author

Guzz-T commented Dec 30, 2025

Rebased branch onto main -> ready for review

@kbabioch
Copy link
Collaborator

kbabioch commented Jan 1, 2026

The pull request now contains a large number of changes. However, all of them have been added based on new findings from the documentation. Would you prefer to split these changes across several pull requests?

Nah, for me its fine to proceed here. You've split it into multiple commits anyway.

@kbabioch
Copy link
Collaborator

kbabioch commented Jan 1, 2026

Overall looks good to me, happy to merge this. Have started some (small) discussions for some lines (see above), but no big strategic topics, etc.

@kbabioch kbabioch self-requested a review January 3, 2026 07:14
@kbabioch kbabioch self-assigned this Jan 3, 2026
@kbabioch kbabioch merged commit d0ea85b into Bouni:main Jan 3, 2026
4 checks passed
@Guzz-T Guzz-T deleted the try/shi-fields branch January 3, 2026 15:09
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.

2 participants