-
Notifications
You must be signed in to change notification settings - Fork 2
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
.gz from Data Registry displays error #353
Comments
From Sep 16 call, I think Quinta was going to propose the text for the error message, for OCP to review – assuming this is about inconsistent data types across different releases. |
My thinking is that we might need additional processing steps in Kingfisher Process to perform corrections for known problems (e.g. normalizing a field so that it is always an array, or always an object, or always a literal value). Like for other steps (like In the meantime, we just need a clear error message, prompting the user to contact us, so that we can address the error. We should also log something, so that we can check the logs for which datasets encountered this error. FYI @yolile |
@jpmckinney The proposed mesage is as follows:
In case we are able to define the source of the issue, we offer to add one more line specifying it in the message:
|
Currently, the flatten tool is failing for all the datasets that I tried from the data registry. (eg https://flatten.open-contracting.org/#/upload-file?lang=es&url=6f6b59d0-b2e5-4641-94f2-d37b2da5c13a, https://flatten.open-contracting.org/#/upload-file?lang=es&url=a8409b70-b4e3-4d55-a919-9d51d57d292f) The logged error is the same as the one reported in open-contracting/spoonbill#195 |
Looks good, just change the second version to end with |
/data/storage/exporter_dumps/australia_nsw/155/full.jsonl.gz - flatten tool complains about the file format
The text was updated successfully, but these errors were encountered: