search

nlextract / NLExtract

Convert (ETL) and visualize free Dutch geo-datasets.
https://nlextract.nl
GNU General Public License v3.0
153 stars 83 forks source link

NLExtract Docker #293

Closed Spijkervet closed 3 years ago

Spijkervet commented 3 years ago

I could not find a fully working Dockerised example of NLExtract, so I created my own pipeline that:

See DOCKER.md for instructions.

webpatser commented 3 years ago

super cool. alleen werkt 1 script niet goed:

docker exec -i -t postgis /bin/bash -c "psql --user postgres -d bag < /app/bag/db/script/adres-tabel.sql"

mis ik een stap ervoor?

justb4 commented 3 years ago

@webpatser de Docker Volume mapping mist: -v "${PWD}:/app", zie ook de README bij deze PR.

Dan wordt het m.i.: docker exec -v "${PWD}:/app" -it postgis /bin/bash -c "psql --user postgres -d bag < /app/bag/db/script/adres-tabel.sql"

justb4 commented 3 years ago

@Spijkervet we zijn bezig met eerst een paar andere PRs, vooral #291. Dan is de hele repo naar Python3. Deze Docker bijdrage is alleen voor BAG? De andere basisregistraties gebruiken Stetl. Daar is een Docker image voor: geopython/stetl. Die zou ook zomaar kunnen werken.

Spijkervet commented 3 years ago

@justb4 Yes, dit is alleen maar voor de BAG. Die had ik nodig voor een huizenwaarde bedrijfje waarmee ik bezig ben :) De rest zou inderdaad eenvoudigweg met stetl kunnen worden gebruikt - zoals je ziet in de Docker.MD is het vrij eenvoudig een postgis of andere Docker container aan te maken waarmee het NLExtract python interface werkt.

Spijkervet commented 3 years ago

Oja - wellicht ten overvloede - deze Docker contribution is dus ook alleen voor Python 3. Misschien dus handig dat jullie eerst #291 afhandelen.

justb4 commented 3 years ago

Ik sluit deze:

De huidige Dockerfile is nog niet voor de overige basis-registraties naast BAG v2 uitgeprobeerd!