Skip to content

Add preliminary calculations fields for 252, 259 and 260#262

Open
Guzz-T wants to merge 2 commits intoBouni:mainfrom
Guzz-T:issue/180/calculations
Open

Add preliminary calculations fields for 252, 259 and 260#262
Guzz-T wants to merge 2 commits intoBouni:mainfrom
Guzz-T:issue/180/calculations

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Mar 15, 2026

This extension does not affect existing queries on the fields used to date. It simply adds new descriptions that can be queried in addition to the existing ones.

Relates to #180

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __main__.py25250%3–66
   datatypes.py450199%147
   discover.py58198%63
luxtronik/cfi
   interface.py175398%167–168, 256
luxtronik/definitions
   __init__.py208199%216
luxtronik/scripts
   __init__.py35586%7–16, 51
   discover_shi.py60600%8–98
   dump_cfi.py14657%18–25, 29
   dump_shi.py15660%21–28, 32
   performance_shi.py63630%7–89
   update_screen.py1058222%6, 20–41, 46–48, 51–61, 64, 70–75, 78, 81–85, 88–92, 95–97, 100–103, 106–111, 115–116, 119–126, 129–136
   watch_cfi.py321650%36–47, 50–56, 60
   watch_shi.py331652%38–49, 52–58, 62
luxtronik/shi
   interface.py337399%438, 519, 527
TOTAL243228888% 

Tests Skipped Failures Errors Time
378 0 💤 0 ❌ 0 🔥 18.162s ⏱️

"writeable": False,
"datatype": 'UINT32',
"unit": '',
"description": 'TODO: Provisional new description based on #180',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want to merge it like this, or this this something you want to do before merging it?

Copy link
Contributor Author

@Guzz-T Guzz-T Mar 15, 2026

Choose a reason for hiding this comment

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

I’d like to wait for a reply from @dasSofa to see if the description is correct. Then I’ll merge it and remove the TODO from the description later if everything is in ok and the open points from #180 are done.

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