Closed xivind closed 4 months ago
Må håndtere blank moving_time
Se samtidig på funksjonen for regning av tid, division by zero
Se samtidig på funksjonen for regning av tid, division by zero
Forsøkt løst med 99e66f3 Dette gjelder tidsfunksjonen i presentator
Klar for review!
Sannelig! Feilkontrollen var på plass, men i mellomtiden har jeg introdusert en annen feil et annet sted (#2 ) som gjør at scriptet crasher hver gang. Trodde det var så enkelt at jeg ikke trengte å teste. Gjør ikke den feilen igjen :) Nå er det hvert fall fikset med 0670d41 Klar til review igjen!
Kjører fint nå! Skal sjekke om det også kjører med flere klubber i input.
Ser ut til å fungere fint, resultatet ligger i test branch, så den bør nok ikke merges med main for da får vi plutselig inn mange resultater vi ikke vil ha med :-)
Kult at det funket nå! Det vil si det, crasher hvert fall ikke, men optimalt er det ikke, ettersom det ser ut som de som er hentet fra sykkel-klubber jukser, siden tiden står til 00:00. Uansett, enig med deg, denne issuen er løst og vel så det. Så får vi si som et krav at koden forutsetter multisport.
Vi ønsker å vite om noe har crashet, og siden det ikke logges noe til konsoll, kan vi lage noe enkelt som skriver resultat fra kjøringen og evt feil til en loggfil.