hdir / strava-club

Web-scraping tool and presentation of club results on github pages. Summarized results for a period of time and counting tickets for competition prizes.
https://hdir.github.io/strava-club/vaar24
2 stars 0 forks source link

Nye statistikk felter og avrunding av aggregerte data #54

Closed xivind closed 2 months ago

xivind commented 2 months ago

@thomiz tar du inn denne om du synes det ser greit ut?

thomiz commented 2 months ago

Når jeg testkjører får jeg dette: image

xivind commented 2 months ago

Ah.. Den &nbsp skal egentlig oversette til mellomrom. Det funket når jeg så på den i ren markdown, men tydeligvis ikke i HTML. Titter på det når jeg kommer ned fra fjellheimen.

lør. 13. apr. 2024, 19:22 skrev Thomas Tveit Rosenlund < @.***>:

Når jeg testkjører får jeg dette: image.png (view on web) https://github.com/hdir/strava-club/assets/6229665/c81e1a7f-45c0-41f0-b139-a467f09ccebf

— Reply to this email directly, view it on GitHub https://github.com/hdir/strava-club/pull/54#issuecomment-2053707760, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACESDY2Z56R4ONPFXBWCTG3Y5FSWVAVCNFSM6AAAAABGFJD5KKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJTG4YDONZWGA . You are receiving this because you authored the thread.Message ID: @.***>

thomiz commented 2 months ago

Fungerer det ikke med to tabellkolonner @xivind? En for ganger rundt jorden og en for lindesnes-nordkapp?

xivind commented 2 months ago

Joda, det vil funke, og det var det jeg prøvde først. Det så litt rart ut fordi siste raden da bare fikk to celler som ikke dekket hele tabellens lengde. Men jeg kan sette det opp slik igjen. Eventuelt kan vi prøve med colspan="2" på den cellen som har mest tekst, så vil de to cellene i sum dekke hele tabellens lengde. Jeg fikser når jeg kommer hjem etterpå. Eventuelt om du har tid, så er det denne blokken her som må justeres. Men jeg fikser gjerne i kveld.

Jeg tror ny kode skal se slik ut, men får ikke testet fra terminalen jeg bruker nå. `

{round(int(self.results.aggregated_summary['distance'])/2318, 1)} ganger Lindesnes - Nordkapp
                <td style="text-align: center; font-size: 19px">
                <img src="assets/images/verden.png" style="height: 25px;">
                {round(int(self.results.aggregated_summary['distance'])/40075, 1)} ganger rundt jorden
                </td></tr>`
thomiz commented 2 months ago

Der tror jeg vi har det https://hdir.github.io/strava-club/TestDev/

image

Hvis det er OK, så pusher jeg endringen til main @xivind

xivind commented 2 months ago

Dette så veldig bra ut @thomiz Tommel opp for å push til main, i grevens tid til aksjonsstart. Smart måte du løste det på ved å ta colspan="2" på de andre statistikkfeltene 👍👍