Skip to content

Move the outdated dictionaries next to the definition dictionaries#239

Open
Guzz-T wants to merge 1 commit intoBouni:mainfrom
Guzz-T:issue/221/outdated
Open

Move the outdated dictionaries next to the definition dictionaries#239
Guzz-T wants to merge 1 commit intoBouni:mainfrom
Guzz-T:issue/221/outdated

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Feb 16, 2026

The outdated definitions belong more to the field definitions.

Relates to #221

@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __main__.py25250%3–66
   collections.py125199%304
   data_vector.py53198%125
   datatypes.py399299%57, 127
   discover.py58198%63
luxtronik/cfi
   interface.py1661392%80–93, 242–243, 248
   parameters.py31197%55
luxtronik/definitions
   __init__.py237199%172
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
   contiguous.py121298%224–225
TOTAL244630188% 

Tests Skipped Failures Errors Time
380 0 💤 0 ❌ 0 🔥 35.877s ⏱️

@kbabioch
Copy link
Collaborator

What is supposed to happen with those outdated definitions?

@Guzz-T
Copy link
Contributor Author

Guzz-T commented Feb 17, 2026

I would leave the reference to the new field there for at least one release.

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