Skip to content

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Jan 6, 2026

No effective code change!

Only classes and functions were moved and the corresponding imports were changed.
Everything related to the socket interface has been moved to the subfolder cfi (config interface).

Changes:

  • LuxtronikData and LuxtronikSocketInterface from luxtronik/__init__.py to luxtronik/cfi/interface.py
  • Socket constants from luxtronik/constants.py to luxtronik/cfi/constants.py
  • parse_version and version_in_range from luxtronik/shi/common.py to luxtronik/common.py
  • LuxtronikDefinition, LuxtronikDefinitionsDictionary and LuxtronikDefinitionsList from luxtronik/shi/definitions.py to luxtronik/definitions/__init__.py
  • The unit test files were also modified.

Follow-up of #222, relates to #221

@Guzz-T Guzz-T changed the title Restructure source code and imports. Restructure source code and imports. Jan 6, 2026
@Guzz-T Guzz-T changed the title Restructure source code and imports. Restructure source code and imports Jan 6, 2026
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __main__.py25250%3–62
   datatypes.py399299%57, 127
   discover.py58198%62
luxtronik/cfi
   interface.py1501391%82–95, 244–245, 250
   parameters.py29197%46
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
TOTAL223230986% 

Tests Skipped Failures Errors Time
363 0 💤 0 ❌ 0 🔥 14.399s ⏱️

@kbabioch kbabioch merged commit 548050a into Bouni:main Jan 6, 2026
4 checks passed
@Guzz-T Guzz-T deleted the issue/221/common branch January 6, 2026 22:54
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