From f5c79e8767dd336387035bbe1af47aab8ed12d31 Mon Sep 17 00:00:00 2001 From: Erol Haagenrud Date: Tue, 18 Aug 2026 09:06:45 +0200 Subject: [PATCH] =?UTF-8?q?ADR-084/CHANGELOG:=20Tj=C3=B8me=20reimportert?= =?UTF-8?q?=20i=20teeoff=5Fcourse=5Fcoordinate=20ogs=C3=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fant en andre, atskilt kopi av Tjømes koordinatdata (den org-tilknyttede "official" banen brukt av turneringer/lag-matchplay) som forrige rundes reimport ikke traff. Samme datasett satt inn der også -- ren datafiks, ingen kodeendring. Co-Authored-By: Claude Sonnet 5 --- ARCHITECTURE_DECISIONS.md | 45 +++++++++++++++++++++++++++++++++++++++ CHANGELOG.md | 28 ++++++++++++++++++++++++ 2 files changed, 73 insertions(+) diff --git a/ARCHITECTURE_DECISIONS.md b/ARCHITECTURE_DECISIONS.md index dba7877..cdfcbea 100644 --- a/ARCHITECTURE_DECISIONS.md +++ b/ARCHITECTURE_DECISIONS.md @@ -8704,6 +8704,51 @@ INSERT 194, én transaksjon) kjørt mot ekte `teecup_db`, deretter bekreftet `side_fairway='left'` etter reimport (matcher kildedataen, rettet fra den kjente `center`-feilen). +### ADR-084 del 7: Tjøme fantes i to atskilte tabeller -- kun én reimportert (2026-08-18) + +Bruker rapporterte at en gruppert hindring fortsatt viste to ikoner -- +undersøkelse viste at forrige rundes reimport (del 6) KUN traff +`golfapi_course_coordinate` (`course_id='0121250146602173'`, brukt av +frittstående "mine runder" via `personal_course.external_golfapi_ +course_id`). Bruker presiserte: "Disse nyeste koordinatene skal brukes +på ALLE Tjømebaner i systemet." + +Full gjennomgang (kun lesende spørringer) fant nøyaktig ÉN til: en +"official"-bane i `course`-tabellen ("Tjøme Golfklubb – Hovedbanen", +`external_course_ref='tjome-golfklubb:140'`), med tilhørende data i +`teeoff_course_coordinate` -- brukt av org-turneringer og lag- +matchplay (`hole_history.py` sin `CourseKey`-oppløsning, IKKE +frittstående runder). Denne hadde fortsatt de gamle 180 punktene: 0 +`hazard_group`, samme `center`-fallback-feil, ingen `creek`/`fairway`/ +`stone_fence`. Ingen tredje representasjon funnet (bekreftet via +`course`/`golfapi_course`/`personal_course` -- alle Tjøme-oppføringer +listet eksplisitt). + +**Løsning:** samme 194-rads datasett fra del 6 (identisk kilde, samme +Python-genererte rader) satt inn i `teeoff_course_coordinate` i stedet +-- kolonnene er identiske (begge tabeller delte `course_poi_type`/ +`course_poi_location`/`course_poi_side`-domenene siden migrasjon 079). +Egen generert reimport-SQL (`DELETE`+`INSERT`, én transaksjon, samme +mønster som del 6). + +**Verifisert:** kjørt FØRST mot en scratch-kopi av de ekte 180 radene +fra `teeoff_course_coordinate` -- 194/39 riktig, identisk resultat som +del 6. Ingen ny visuell scratch-runde (identisk radstruktur/kodevei +allerede bekreftet flere ganger). + +**Rullet ut 2026-08-18** -- bruker bekreftet. Reimport-SQL (DELETE +180/INSERT 194, én transaksjon) kjørt mot ekte `teecup_db`. Bekreftet +194/39/`poi_type`-fordeling identisk med `golfapi_course_coordinate`, +hull 7 sitt vann `side_fairway='left'`. Ingen kode-/frontend-endring +denne delen (samme deployerte kode fra del 6 leser nå riktig data fra +begge kilder). + +**Lærdom:** Tjøme har to helt separate koordinat-datasett i systemet +(én for frittstående runder, én for org-turneringer) -- verdt å huske +ved FREMTIDIGE data-korrigeringer for denne (eller andre) baner med +begge tilkoblingstyper. Ikke dokumentert eksplisitt noe sted fra før; +bør vurderes nevnt i CLAUDE.md om det skjer igjen. + --- ## Utviklingsplan (rekkefølge) diff --git a/CHANGELOG.md b/CHANGELOG.md index 49f5161..15659b5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12277,3 +12277,31 @@ Neste steg: 194, én transaksjon) kjørt mot ekte `teecup_db`, deretter `docker compose build teecup_frontend && up -d`. Hull 7 sitt vann bekreftet `side_fairway='left'` etter reimport. + +107. **Tjøme fantes i to atskilte tabeller -- kun én reimportert + (ADR-084 del 7) — 2026-08-18.** Bruker rapporterte at en gruppert + hindring fortsatt viste to ikoner -- viste seg at forrige rundes + reimport kun traff `golfapi_course_coordinate` (frittstående + runder). Fant én til: `teeoff_course_coordinate` for + `external_course_ref='tjome-golfklubb:140'` (den "official" + org-tilknyttede banen "Hovedbanen", brukt av turneringer/lag- + matchplay) -- fortsatt med de gamle 180 punktene, 0 hazard_group. + Ingen tredje representasjon funnet (bekreftet via `course`/ + `golfapi_course`/`personal_course`). + + Samme 194-rads datasett fra forrige runde satt inn i + `teeoff_course_coordinate` i stedet -- identiske kolonner/domener + siden migrasjon 079. Kjørt FØRST mot en scratch-kopi av de ekte + 180 radene, 194/39 bekreftet identisk med forrige runde. Ingen + kode-endring nødvendig (samme deployerte kode leser nå riktig + data fra begge kilder). + + **Rullet ut 2026-08-18** -- bruker bekreftet. Reimport-SQL (DELETE + 180/INSERT 194, én transaksjon) kjørt mot ekte `teecup_db`. + Bekreftet 194/39/poi_type-fordeling identisk med forrige runde, + hull 7 sitt vann `side_fairway='left'`. + + **Lærdom:** Tjøme har to helt separate koordinat-datasett (én for + frittstående runder, én for org-turneringer) -- verdt å huske ved + fremtidige datakorreksjoner, for denne eller andre baner med + begge tilkoblingstyper.