hubmapconsortium / ingest-api

MIT License
0 stars 0 forks source link

Update metadata validation to work with the latest IVT #561

Closed yuanzhou closed 1 month ago

yuanzhou commented 1 month ago

From Max:

The IVT was just updated on main with commit b42126ea3fae02dfd876415b1d47d02534864d8d. This has the fix for the row numbers in the error reporting

This is the function validate_tsv() that will have to be updated in the HM Ingest API to support the updated error reporting: https://github.com/sennetconsortium/ingest-api/blob/2fdc209ca70ce5ed550a0036d8fa82d60525144e/src/jobs/validation/metadata.py#L167-L214

The call for get_schema_version has been updated and now get_tsv_errors just returns a flat array of errors so you don’t have to parse them out