search

datosgobar / pydatajson

Librería para analizar, generar y validar metadatos en formato data.json.
https://pydatajson.readthedocs.io
MIT License
14 stars 9 forks source link

Arreglar lectura de catálogos en formato xlsx que no son válidos #270

Open FScaccheri opened 5 years ago

FScaccheri commented 5 years ago

Específicamente, cuando un catálogo en formato xlsx no tiene distribution_identifier

abenassi commented 5 years ago

Franco, cuál sería este caso? No se me ocurre ningún catálogo en XLSX que no tenga "distribution_identifier", o si no lo tiene debería reportarse el error y fallar.

El jue., 1 ago. 2019 a las 16:13, Franco Scaccheri (< notifications@github.com>) escribió:

Específicamente, cuando un catálogo en formato xlsx no tiene distribution_identifier

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/datosgobar/pydatajson/issues/270?email_source=notifications&email_token=ABBTQYXGC7MYYLYUJWB5RBTQCMYVTA5CNFSM4IITUQO2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HC42AGA, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBTQYR275IKB25FW7Y55ALQCMYVTANCNFSM4IITUQOQ .

FScaccheri commented 5 years ago

Dentro de los samples en el repo de pydatajson, en la carpeta de catálogos válidos, me descargué este para probar algunos features localmente: https://github.com/datosgobar/pydatajson/blob/master/samples/archivos-tests/excel-validos/catalogo-justicia.xlsx Al probar, por ejemplo, levantarlo con pydatajson o cargarlo y validarlo en mi instancia local de infra, me tira un KeyError al querer leer el distribution_identifier, que si abro el archivo y busco, veo que efectivamente no está

abenassi commented 5 years ago

Ahh entonces ese catálogo ya no es válido, quedó mal clasificado del pasado cuando no había distribution_identifier me parece

El jue., 1 ago. 2019 a las 16:44, Franco Scaccheri (< notifications@github.com>) escribió:

Dentro de los samples en el repo de pydatajson, en la carpeta de catálogos válidos, me descargué este para probar algunos features localmente:

https://github.com/datosgobar/pydatajson/blob/master/samples/archivos-tests/excel-validos/catalogo-justicia.xlsx Al probar, por ejemplo, levantarlo con pydatajson o cargarlo y validarlo en mi instancia local de infra, me tira un KeyError al querer leer el distribution_identifier, que si abro el archivo y busco, veo que efectivamente no está

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/datosgobar/pydatajson/issues/270?email_source=notifications&email_token=ABBTQYUVVI4OT5ALPMNWG53QCM4LPA5CNFSM4IITUQO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3LVXSQ#issuecomment-517430218, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBTQYTVCKGJCCB5SLDY6W3QCM4LPANCNFSM4IITUQOQ .