search

jolpica / jolpica-f1

Apache License 2.0
41 stars 1 forks source link

Sprint weekends contain incorrect additional session on races endpoint #58

Closed theOehrly closed 1 month ago

theOehrly commented 1 month ago

Describe the bug

51 attempted to fix the representation of sprint weekends in the races endpoint. While the "SprintQualifying" entry now exists, the "SecondPractice" entry still exists. Sprint weekends do not have a second practice, though. It seems like this got lost during refactoring.

To Reproduce http://localhost:5000/ergast/f1/2024/5.json

Expected behavior The entry for second practice should not be included.

Screenshots grafik

jolpica commented 1 month ago

I think this is actually intended functionality, but an issue with the data.

Looks like the import script creates a practice 2 and a sprint quali.

image

This ticket can be to fix the underlying data.