Unify and harden add_stats results parsing #102

Merged
pecheny merged 3 commits from land-pr-2-add-stats into main 2026-05-31 22:00:13 +00:00
Owner

Refactor xlsx_to_results and custom_csv_to_results onto a shared
_results_table_to_masks core that handles both the per-tour and
non-tour rating.chgk.info layouts for xlsx and csv alike (including a
UTF-8 BOM and leading blank rows in csv exports).

  • use the module logger instead of print
  • treat unresolved controversials / unexpected cell values as not taken
    and warn, instead of aborting the whole file
  • add tests covering both xlsx layouts, csv layouts, custom delimiters,
    disputed values and malformed input
Refactor xlsx_to_results and custom_csv_to_results onto a shared _results_table_to_masks core that handles both the per-tour and non-tour rating.chgk.info layouts for xlsx and csv alike (including a UTF-8 BOM and leading blank rows in csv exports). - use the module logger instead of print - treat unresolved controversials / unexpected cell values as not taken and warn, instead of aborting the whole file - add tests covering both xlsx layouts, csv layouts, custom delimiters, disputed values and malformed input
Author
Owner

mentioned in commit c7ce465004

mentioned in commit c7ce465004547c948b13bf52f082f5b758892541
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
pecheny/chgksuite!102
No description provided.