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

First attempt to dockerize scripts execution #13

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

jenkin
Copy link
Contributor

@jenkin jenkin commented Dec 18, 2019

  • python main.py works
  • bash _utils/analisiGeometrie.sh doesn't works

Shortcuts: bash build.sh && bash run.sh.

- `python main.py` works
- `bash _utils/analisiGeometrie.sh` doesn't works
@jenkin
Copy link
Contributor Author

jenkin commented Dec 18, 2019

L'immagine di base è python:3.7-slim, basata a sua volta su debian buster.

Ecco l'errore nell'esecuzione dello script che corregge le geometrie (vedi #12): Error: no such function: ST_MakeValid, idem per la funzione MakeValid. @aborruso puoi darci un'occhiata?

Qui le librerie di sistema installate.

@jenkin
Copy link
Contributor Author

jenkin commented Jan 9, 2020

@aborruso ora la correzione delle geometrie istat gira correttamente sotto docker, puoi verificarne il risultato nelle varie cartelle in 20190101? In zip ci sono i file originali istat, in shp quelli corretti, tutti gli altri formati derivano da quelli corretti. Grazie!

@aborruso
Copy link

aborruso commented Jan 9, 2020

@jenkin al momento non posso fare il test, a partire da pipenv install.

Quindi o mi invii tutti i file di output di 20190101, o produci l'ouput nel repo. Sorry

@aborruso
Copy link

aborruso commented Jan 9, 2020

@jenkin ritiro quanto detto, hai messo tutto nel branch https://github.com/teamdigitale/confini-amministrativi-istat/tree/feature/dockerization

grazie

@jenkin
Copy link
Contributor Author

jenkin commented Jan 10, 2020

Sì, prima del merge aggiornerò il README con tutte le istruzioni per far girare lo script...

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