Repository navigation
feat: add new race results - #312
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
โน๏ธ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with ๐ while any review is running, comments if it has suggestions, and reacts with ๐ once all reviews finish with no findings. |
There was a problem hiding this comment.
๐ก Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4ca26af0c2
โน๏ธ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with ๐.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -0,0 +1,1063 @@ | |||
| FirstName,LastName,FullName,Bib,AgeGroup,Gender,City,State,Country,CountryISO,SwimTime,BikeTime,RunTime,T1Time,T2Time,FinishTime,SwimSeconds,BikeSeconds,RunSeconds,T1Seconds,T2Seconds,FinishSeconds,OverallRank,GenderRank,AgeGroupRank,SwimRankOverall,SwimRankGender,SwimRankGroup,BikeRankOverall,BikeRankGender,BikeRankGroup,RunRankOverall,RunRankGender,RunRankGroup,Status,Points | |||
| Amandin,Colvray,Amandin Colvray,4,M30-34,Male,cusset,,France,FR,0:24:52,2:07:15,1:17:15,0:02:04,0:01:50,3:53:16,1492,7635,4635,124,110,13996,1,1,1,7,7,2,2,2,1,1,1,1,Finisher,3500 | |||
| Aziz,Haddaj,Aziz Haddaj,821,M40-44,,ุฅูุฒูุงู,Souss-Massa,Morocco,MA,0:28:31,2:13:05,1:15:07,0:03:31,0:02:05,4:02:18,1711,7985,4507,211,125,14538,2,2,1,40,39,3,9,9,1,2,2,1,Finisher,3500 | |||
There was a problem hiding this comment.
Normalize missing genders before publishing results
When a finisher's contact gender is absent, as for Aziz Haddaj here despite M40-44 and GenderRank=2, the published race data leaves Gender empty. computeRaceStats only includes exact Male/Female values in leaderboards, and segmentLabels drops empty genders, so the second-place man is omitted from the men's leaderboard and all gender-filtered distributions; the new datasets contain 26 such finishers, plus several contradictory gender/age-group values. Derive or validate gender from the age-group prefix before generating the CSV and indexes.
Useful? React with ๐ย / ๐.
Summary
Automated daily scrape discovered new race results.
New files
data/im703-agadir-2026.csvdata/im703-greece-2026.csvdata/im703-new-york-2026.csv