Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update layers file with fresh data from the NGR spider (+ add BGT OGC API Features). #118

Merged

Conversation

rkettelerij
Copy link
Contributor

In particular using the functionality from PDOK/ngr-services-spider#31 which includes the new BGT OGC API Features.

In particular using the functionality from PDOK/ngr-services-spider#31 which includes the new BGT OGC API Features.
@rkettelerij
Copy link
Contributor Author

rkettelerij commented Jun 11, 2024

Hoi @rduivenvoorde,

Dit is een vervolg op het werk wat @kad-rothul eerder heeft gedaan in #111. Op dat moment was er nog geen OGC API Features service live. Dat is nu wel het geval met de BGT OGC API Features. Die zit nu in deze PR. Verder zijn uiteraard ook alle andere services/layers bijgewerkt met het NGR.

NB: ik heb deze PR gemaakt obv PDOK/ngr-services-spider#31, die is op moment van schrijven nog niet gemerged naar master. Maar dat zal spoedig gebeuren. Daarna zal ik deze PR updaten met de nieuwe spider (in het generate-pdok-layers-config.sh bestand). > Nieuwe ngr-spider is beschikbaar. @rduivenvoorde wat mij betreft klaar voor review.

Als je vragen/opmerking hebt hoor ik het graag.

@rkettelerij rkettelerij changed the title Update layers file with fresh data from the NGR spider. Update layers file with fresh data from the NGR spider (+ add BGT OGC API Features). Jun 11, 2024
@rduivenvoorde
Copy link
Owner

Hoi, ik ga 'm even mergen en dan bekijken.

@rduivenvoorde rduivenvoorde merged commit 732f09f into rduivenvoorde:develop Jun 14, 2024
@rduivenvoorde
Copy link
Owner

@rkettelerij ik zie dat de actuele luchtfoto's niet meer als WMTS beschikbaar zijn, maar (blijkbaar) alleen als WMS.
Weet jij hier iets van? Het lijkt zeer performend, dus misschien is dat de reden?

Ik zie inderdaad de BGT OGC API - Features lagen werkend. ( ook even @raymondnijssen )
Hoe belangrijk zie jij deze voor de gebruiker? Eigenlijk is de vraag: moeten we die opnemen in het script zoals we ook doen voor de luchtfoto's enzo: dus dat ze prominenter bovenaan staan voor gebruikers?

Verder wil ik graag nog even een bugje wegwerken voordat ik een nieuwe release maak:
#119

@rkettelerij
Copy link
Contributor Author

rkettelerij commented Jun 17, 2024

Dank voor de merge.

@rkettelerij ik zie dat de actuele luchtfoto's niet meer als WMTS beschikbaar zijn, maar (blijkbaar) alleen als WMS. Weet jij hier iets van? Het lijkt zeer performend, dus misschien is dat de reden?

De WMTS zijn er nog, en dit is ook de gewenste ingang voor afnemers. In het layers bestand zie ik de services nog wel voorkomen. Ook in QGIS staan ze in de lijst (van de plugin).

luchtfoto wmts

Wat wel opvalt is dat het NGR dataset record is gewijzigd en nu een fout geeft. Zie bijv. Luchtfoto Actueel Ortho 8cm RGB WMTS maar het komt bij alle luchtfoto WMTS'en voor. Dus heel goed dat je dit hebt gesignaleerd! Ik heb het inmiddels intern doorgezet en hopelijk wordt dit snel opgelost.

Ik zie inderdaad de BGT OGC API - Features lagen werkend. ( ook even @raymondnijssen ) Hoe belangrijk zie jij deze voor de gebruiker? Eigenlijk is de vraag: moeten we die opnemen in het script zoals we ook doen voor de luchtfoto's enzo: dus dat ze prominenter bovenaan staan voor gebruikers?

De BGT Vector Tiles staan nu ook bovenaan. Als het mogelijk is om de BGT Features ook in die regio op te nemen zou dat heel mooi zijn! PDOK ziet OGC APIs ook als strategisch belangrijk. Voor de BGT is er in het verleden wel om een WFS gevraagd maar dit bleek om o.a. vanwege schaalbaarheid/performance redenen niet goed mogelijk. Met de OGC API Features zijn individuele features (incl. volledige historie) nu wel beschikbaar gekomen. Dus in die zin is het denk ik voor gebruikers ook relevant.

Verder wil ik graag nog even een bugje wegwerken voordat ik een nieuwe release maak: #119

👍

@rkettelerij
Copy link
Contributor Author

@rduivenvoorde naar de luchtfoto's wordt gekeken, ik geeft een seintje als het weer correct is.

rkettelerij added a commit to rkettelerij/pdokservicesplugin that referenced this pull request Jun 20, 2024
@rkettelerij
Copy link
Contributor Author

@rduivenvoorde issue omtrent de luchtfoto records/services is opgelost. Zat in NGR zelf. Ik heb een nieuwe PR gemaakt met bijgewerkte layers: #120 (of je kan zelf het script draaien).

@rkettelerij rkettelerij deleted the update-with-ogc-api-features branch June 20, 2024 09:27
@rduivenvoorde
Copy link
Owner

@rkettelerij ik draai 'm meestal zelf, omdat ik dan ook nog wat zaken kan checken. Daarnaast voeg ik handmatig zoiezo nog de OpenBasisKaart toe (als alternatief voor de OpenTopo).
Ik ga er een dezer dagen even mee aan de gang.

@rduivenvoorde
Copy link
Owner

Ok, ik heb (develop-branch) een versie 0.5.1 klaar staan!
Belangrijkste veranderingen: fix voor #119 en de ogc-api-features toevoegingen ( en een volledig geupdate pdok-layers.json dus)

@rkettelerij @raymondnijssen zouden jullie voordat ik 'm release even de develop branch willen testen?

@rkettelerij
Copy link
Contributor Author

@rduivenvoorde getest en wat mij betreft ok 👍

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.

2 participants