search

transparencia-mg / dtamg-py

MIT License
0 stars 0 forks source link

Adicionar argumento em `dtamg-py etl-make` para controlar qual data package deve ser utilizado #17

Open fjuniorr opened 2 years ago

fjuniorr commented 2 years ago

O objetivo é permitir a utilização tanto do datapackage.yaml quanto do datapackage.json.

Como o datapackage.json é o data package pela especificação, ele deve ser o default.

fjuniorr commented 2 years ago

@gabrielbdornas se https://github.com/frictionlessdata/frictionless-py/issues/221 for implementado essa função perde a razão de ser.

Será que a gente tenta contribuir diretamente upstream?

fjuniorr commented 2 years ago

@gabrielbdornas o PR https://github.com/frictionlessdata/frictionless-py/pull/1112 com essa funcionalidade já está na main, falta apenas sair uma nova versão.

fjuniorr commented 2 years ago

Funcionalidade disponível a partir da versão v4.39.0 do frictionless-py e utilizada no projeto da remuneração.

gabrielbdornas commented 2 years ago

@fjuniorr, não deveríamos excluir a função validate daqui e publicar uma nova versão do dtamg?

https://github.com/transparencia-mg/dtamg-py/blob/6007d33a1f33f01f504f82a52ed298cc9f466f61/dtamg_py/utils.py#L191-L195

Podemos fazer a mesma manutenção feita na remuneração no age7, o que acha?

https://github.com/transparencia-mg/age7/blob/b8b99dc8607ea84b14c6fc8a64ee4c7c7bf49311/Makefile#L88-L89

Por fim, acho que não entendi bem a relação do título deste issue com a nova funcionalidade frictionless da versão 4.39 de validar recursos específicos.

fjuniorr commented 2 years ago

@fjuniorr, não deveríamos excluir a função validate daqui e publicar uma nova versão do dtamg? [...] Podemos fazer a mesma manutenção feita na remuneração no age7, o que acha?

Faz sentido sim, pro comportamento ficar equivalente.

Por fim, acho que não entendi bem a relação do título deste issue com a nova funcionalidade frictionless da versão 4.39 de validar recursos específicos.

É pq esse issue foi criado especificamente pro uso da validação na remuneração usando o datapackage.json e não o datapackage.yaml com fonte da verdade.