feat: visa antal exkluderade rundor i player history-accordion #21

Closed
opened 2026-05-25 08:17:12 +02:00 by shcizo · 0 comments
Owner

Beskrivning

Predicted rating-algoritmen exkluderar rundor som faller under både 2.5σ och 100-poäng-cutoff (aktiveras vid ≥7 rundor i 12-månadersfönstret). Idag är denna info gömd bakom en "View calculation details"-modal som visar hela debugLog som monospace-text.

Vi vill istället visa antalet exkluderade rundor direkt som en rad i accordion-grid:en, och ta bort modalen.

Design

Lägg till en ny rad sist i <dl class="detail-grid"> (se views/partials/player-history.ejs:8-39):

Exkluderade rundor: 2
  • Visa alltid raden (även "0 exkluderade" när filtret inte aktiverats eller inga rundor föll utanför)
  • Samma layout som befintliga dt/dd-par ("Round spread", "Rating range")
  • Samma rad ska in i mobile-kortet (views/partials/ratings-cards.ejs:118-125)

Tekniska noteringar

Datakälla — debugLog innehåller redan informationen:

  • src/services/rating-calculator.js:163-177 loggar varje exkluderad runda med datum, competition och vilket filter som triggade
  • Räkna entries med excluded-flagga (eller motsvarande markör — se debugLog-formatet) för att få siffran

Persistens:

  • player.stdDev lagras redan i DB efter beräkning — lägg till player.excludedRoundsCount på samma sätt
  • Sätts vid POST /api/refresh-round-history/:pdgaNumber (src/routes/players.js) tillsammans med rating och stdDev
  • Migration i src/db.js för ny kolumn

Modal-borttagning:

  • Ta bort showDebugInfo() från public/js/players.js
  • Ta bort "View calculation details"-knappen i både player-history.ejs:40 och ratings-cards.ejs
  • Ta bort eventuella oanvända debugLog-fält från API-responsen om de inte längre används någon annanstans

Scope

Ingår:

  • Räkna och persistera antal exkluderade rundor
  • Visa siffran i desktop accordion + mobile cards
  • Ta bort modal-funktionalitet

Ingår inte:

  • Lista över vilka rundor som exkluderats (bara antalet)
  • Visa cutoff-värdet (mean - 100 eller mean - 2.5σ)
  • Förklarande text om när filtret aktiveras
## Beskrivning Predicted rating-algoritmen exkluderar rundor som faller under både 2.5σ och 100-poäng-cutoff (aktiveras vid ≥7 rundor i 12-månadersfönstret). Idag är denna info gömd bakom en "View calculation details"-modal som visar hela debugLog som monospace-text. Vi vill istället visa antalet exkluderade rundor direkt som en rad i accordion-grid:en, och ta bort modalen. ## Design Lägg till en ny rad sist i `<dl class="detail-grid">` (se `views/partials/player-history.ejs:8-39`): ``` Exkluderade rundor: 2 ``` - Visa alltid raden (även "0 exkluderade" när filtret inte aktiverats eller inga rundor föll utanför) - Samma layout som befintliga dt/dd-par ("Round spread", "Rating range") - Samma rad ska in i mobile-kortet (`views/partials/ratings-cards.ejs:118-125`) ## Tekniska noteringar **Datakälla — debugLog innehåller redan informationen:** - `src/services/rating-calculator.js:163-177` loggar varje exkluderad runda med datum, competition och vilket filter som triggade - Räkna entries med `excluded`-flagga (eller motsvarande markör — se debugLog-formatet) för att få siffran **Persistens:** - `player.stdDev` lagras redan i DB efter beräkning — lägg till `player.excludedRoundsCount` på samma sätt - Sätts vid `POST /api/refresh-round-history/:pdgaNumber` (`src/routes/players.js`) tillsammans med `rating` och `stdDev` - Migration i `src/db.js` för ny kolumn **Modal-borttagning:** - Ta bort `showDebugInfo()` från `public/js/players.js` - Ta bort "View calculation details"-knappen i både `player-history.ejs:40` och `ratings-cards.ejs` - Ta bort eventuella oanvända debugLog-fält från API-responsen om de inte längre används någon annanstans ## Scope **Ingår:** - Räkna och persistera antal exkluderade rundor - Visa siffran i desktop accordion + mobile cards - Ta bort modal-funktionalitet **Ingår inte:** - Lista över *vilka* rundor som exkluderats (bara antalet) - Visa cutoff-värdet (mean - 100 eller mean - 2.5σ) - Förklarande text om när filtret aktiveras
shcizo added the enhancement label 2026-05-25 08:17:12 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shcizo/pdga-rating#21