Extract tram links from QGIS, export mbtiles

[jannebe]

This change is 

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[culka]

@culka reviewed 5 files and all commit messages, and made 6 comments.
Reviewable status: all files reviewed, 6 unresolved discussions (waiting on @jannebe).

---

export_mbtiles_tram_links.sh line 11 at r1 (raw file):

DB_TABLE_NAME="tram_links"

MML_TRAM_IMPORT_DATE="2026-01-28"

Tätä ei varmaan pitäsi kovakoodata skriptiin

---

generate-tram-infralinks-from-qgis.py line 2 at r1 (raw file):

# python

Yleisesti ottaen aika kamala rakenne tässä 🙈

Funktioiden määrittelyä satunnaisissa paikoissa filun keskellä, muuttujia joiden arvoja muutellaan ties missä. Ei kuitenkaan refaktoroinnin arvoista tässä käyttötapauksessa.

---

generate-tram-infralinks-from-qgis.py line 28 at r1 (raw file):

import uuid

Mukaan tarkastus joka estää skriptin ajon QGIS:n ulkopuolella.

if qgis.utils.iface is None:
  raise RuntimeError('Script must be run in QGIS')

---

generate-tram-infralinks-from-qgis.py line 142 at r1 (raw file):

        continue

    # 1) Ellipsoidal length in metres on GRS80 using source CRS

Eikös tämä riipu kumpaa koordinaatistoa käytetään? dist tyyppi alustetaan aiemmin.

---

README.md line 275 at r1 (raw file):

2. Open the GeoPackage file in QGIS (drag to Layers)
1. Open Python console

Aika monta ykkössteppiä tässä peräkkäin.

---

set_env.sh line 94 at r1 (raw file):

docker_start() {
  if ! docker container inspect "$DOCKER_CONTAINER_NAME" >/dev/null 2>&1; then
    echo "Docker container $DOCKER_CONTAINER_NAME does not exist. Call docker_run first."

Onko kommentti tässä väärin, vai eikö docker_startilla voi käynnistää dockeria enää? Kaikki vanhat skriptit käyttävät pelkkää docker_startia.

[jannebe]

generate-tram-infralinks-from-qgis.py line 2 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Yleisesti ottaen aika kamala rakenne tässä 🙈

Funktioiden määrittelyä satunnaisissa paikoissa filun keskellä, muuttujia joiden arvoja muutellaan ties missä. Ei kuitenkaan refaktoroinnin arvoista tässä käyttötapauksessa.

Älä ny, AI:n tuottama on aina vaan kaunista, etkö ole kuullut? 😆

[jannebe]

generate-tram-infralinks-from-qgis.py line 28 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Mukaan tarkastus joka estää skriptin ajon QGIS:n ulkopuolella.

if qgis.utils.iface is None:
  raise RuntimeError('Script must be run in QGIS')

Eihän toi edes käynnisty kun ei löydä importtejakaan?

[jannebe]

generate-tram-infralinks-from-qgis.py line 142 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Eikös tämä riipu kumpaa koordinaatistoa käytetään? dist tyyppi alustetaan aiemmin.

poistan "on GSR80 using source CRS"

[jannebe]

set_env.sh line 94 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Onko kommentti tässä väärin, vai eikö docker_startilla voi käynnistää dockeria enää? Kaikki vanhat skriptit käyttävät pelkkää docker_startia.

Näyttäis niissä olevan kaikissa kommentti että oletus on että kontti on olemassa. eli käytännössä, että import_digiroad_shapefiles on ajettu.

[jannebe]

@jannebe made 2 comments.
Reviewable status: all files reviewed, 6 unresolved discussions (waiting on @culka).

---

export_mbtiles_tram_links.sh line 11 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Tätä ei varmaan pitäsi kovakoodata skriptiin

Done.

---

README.md line 275 at r1 (raw file):

Previously, culka (Teemu Mäkinen) wrote…

Aika monta ykkössteppiä tässä peräkkäin.

perus .md. Lukasepa viewerissä ;)

Mutta ehkä se on kiva että perus terminaalissakin näkyy oikein ni avasin noi tohon. 😉

[culka]

@culka reviewed 5 files and all commit messages, and resolved 4 discussions.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @jannebe).