Skip to content

Fix interleaving timetable parse - #113

Open
huppd wants to merge 4 commits into
mainfrom
fix_interleaving_timetable_parse
Open

Fix interleaving timetable parse#113
huppd wants to merge 4 commits into
mainfrom
fix_interleaving_timetable_parse

Conversation

@huppd

@huppd huppd commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator
  • Fix crash on interleaved timing tables: read_logfile now routes each row to the table matching its header's column count (with fallback to earlier headers) instead of assuming rows between two headers all belong to the first one; this previously caused a hard sys.exit(1) crash
  • Unmatched rows now log a warning and are skipped, rather than aborting the whole parse
  • Fixed a related pre-existing bug: the parser was silently dropping the last matching line of every log file; regenerated the affected reference/add JSON fixtures accordingly
  • Added a focused regression test + small dedicated fixture for the interleaving fix, plus a minimal interleaved splice in the existing timing_example_1.txt fixture

This PR were developed with AI assistance (Claude Code). Please review accordingly rather than assuming sole human authorship.

@huppd
huppd marked this pull request as ready for review August 4, 2026 13:38
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.

1 participant