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

Conversión Shapefile a CSV. #47

Merged
merged 14 commits into from
Sep 5, 2017
Merged

Conversión Shapefile a CSV. #47

merged 14 commits into from
Sep 5, 2017

Conversation

arperoni
Copy link
Contributor

@arperoni arperoni commented Sep 4, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 94.311% when pulling 4b81a11 on shp2csv into 48e2d7b on master.

* master:
  Agrega parametros adicionales para controlar el tipo de datos de los campos. Agrega XLSX como formato de entrada soportado.

# Conflicts:
#	data_cleaner/data_cleaner.py
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 21cfbe3 on shp2csv into ** on master**.

…itar problemas de encoding y representación de unicode.
…into shp2csv

* 'shp2csv' of https://github.com/gobabiertoAR/data-cleaner:
  Agrega parametros adicionales para controlar el tipo de datos de los campos. Agrega XLSX como formato de entrada soportado.
* shp2csv:
  Mejoro la conversión de JSON a diccionario y a JSON de nuevo, para evitar problemas de encoding y representación de unicode.

# Conflicts:
#	data_cleaner/data_cleaner.py
#	docs/README.md
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 33988c8 on shp2csv into ** on master**.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling ccfb6b2 on shp2csv into ** on master**.

Copy link
Member

@abenassi abenassi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arperoni estoy mergeando este PR.

Ayer le tuve que agregar algo más que es el tratamiento de la proyección de entrada y salida de un shp, porque resulta que el caso de uso que teníamos que resolver implicaba la necesidad de reconvertir a EPSG 4326 para que se pueda ver en CKAN, y el shp original estaba en coordenadas proyectadas.

Lo agregué un poco de urgencia así que no le escribí tests formales (sólo lo testee), quedan para agregar en tu próximo sprint así de paso te familiarizás con el cambio que hice. Tuve que darle una vuelta al tema para que funcione bien.

@abenassi abenassi merged commit 5499ad6 into master Sep 5, 2017
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b3f1d67 on shp2csv into ** on master**.

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.

None yet

3 participants