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

fix: log Non Visual QA error report in a JSON format #55

Merged
merged 7 commits into from
Jul 27, 2022

Conversation

paulfouquet
Copy link
Collaborator

@paulfouquet paulfouquet commented Jul 15, 2022

Description

In order to facilitate the loggin in Elastic Search, the Non Visual QA error report should be in a JSON format.

Changes

  • Create a dictionnary
{"file": "path/to/the/file", "errors": [{"type": "error_type", "message": "error_message"}]}

Example

{"file": "s3://linz-imagery-staging/RGBi4/tasman_rural_2020_0.3m_a_RGBI/tif/2020_BQ24_5000_0601.tif", "errors": [{"type": "gdalinfo", "message": "error(s): b'Warning 1: The definition of projected CRS EPSG:2193 got from GeoTIFF keys is not the same as the one from the EPSG registry, which may cause issues during reprojection operations. Set GTIFF_SRS_SOURCE configuration option to EPSG to use official parameters (overriding the ones from GeoTIFF keys), or to GEOKEYS to use custom values from GeoTIFF keys and drop the EPSG code.\\n'"}, {"type": "nodata", "message": "noDataValue not set"}, {"type": "bands", "message": "bands count is not 3", "count": "4"}, {"type": "color", "message": "unexpected color interpretation bands", "missing": "band 4 Undefined"}, {"type": "srs", "message": "different srs"}]}

@paulfouquet paulfouquet requested a review from a team as a code owner July 15, 2022 03:26
@paulfouquet paulfouquet marked this pull request as draft July 15, 2022 03:44
@paulfouquet paulfouquet marked this pull request as ready for review July 25, 2022 04:33
scripts/gdal_helper.py Outdated Show resolved Hide resolved
@kodiakhq kodiakhq bot merged commit ed4d623 into master Jul 27, 2022
@kodiakhq kodiakhq bot deleted the fix/non-visual-qa-logs branch July 27, 2022 21:13
@github-actions github-actions bot mentioned this pull request Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants