Skip to content

fix: WFS-kerros valitaan aineiston nimen perusteella - #180

Merged
trotor merged 1 commit into
mainfrom
fix/wfs-kerrosvalinta
Oct 10, 2026
Merged

trotor merged 1 commit into
mainfrom
fix/wfs-kerrosvalinta

Conversation

@trotor

@trotor trotor commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Ongelma

WFS-resurssi on katalogissa usein koko palvelun kyvykkyysosoite. Kerroksettomassa kyselyssä neuvottelu otti kykyjen ensimmäisen kerroksen: "Helsingin kiinteistöt alueina" palautti liikennemäärät, vaikka samassa palvelussa on avoindata:Kiinteisto_alue (kiinteistötunnukset).

Korjaus

  • wfs.choose_layer(feature_types, hints): valitsee kerroksen, jonka nimen sanat vastaavat vihjeitä (perusmuoto, ääkköset taitettuna, CamelCase auki, alkuosa ≥ 5 merkkiä). Täsmälleen sama kerrosnimi vihjeissä ratkaisee suoraan. Ei osumaa → ennallaan (ensimmäinen).
  • Vihjeet (query_source): aineiston otsikko ja muiden resurssien LAYERS/typeName-parametrit.
  • Vastauksen huomautus ja next_actions kertovat, valittiinko kerros nimen perusteella vai ensimmäisenä.

Todennettu oikeilla palveluilla

aineisto ennen jälkeen
Helsingin kiinteistöt alueina Ajoneuvoliikenne_liikennemaarat_viiva Kiinteisto_alue (kiinteistotunnus)
Vantaan rakennukset (ensimmäinen) gis:rakennukset (kiinteistotunnus, osoite)

Espoon Tekla-palvelu ei vastaa kyvykkyyskyselyyn (Palvelu ei kertonut kyvyistään); erillinen, aiempi ongelma.

Testit: 1886 (aura) + 475 (aura-pro) läpi, ruff ja mypy puhtaat.

🤖 Generated with Claude Code

"Helsingin kiinteistöt alueina" palautti palvelun ensimmäisen kerroksen
(liikennemäärät), vaikka samassa palvelussa on avoindata:Kiinteisto_alue.
Kun URL ei nimeä kerrosta, neuvottelu valitsee nyt kerroksen, jonka nimi
vastaa aineiston otsikkoa tai sen muiden resurssien kerrosnimiä
(WMS LAYERS, typeName). Ilman osumaa käytös on ennallaan.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@trotor
trotor merged commit 1d2c388 into main Oct 10, 2026
3 checks passed
@trotor
trotor deleted the fix/wfs-kerrosvalinta branch October 10, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant