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: round footprints to 8 decimal places TDE-1237 #1088

Merged
merged 7 commits into from
Oct 3, 2024

Conversation

amfage
Copy link
Contributor

@amfage amfage commented Sep 23, 2024

Motivation

As a Geospatial Data Engineer I want the geojson files my systems create to contain an appropriate precision for the data being stored so that users aren’t seeing unnecessary data.

Modifications

Use COORDINATE_PRECISION=8 gdal_footprint layer configuration option when creating the footprint files that are used to create the capture area.

Verification

Ran manually. Examples before and after: capture_areas.zip.
Added footprint end-to-end CI test.

@amfage amfage added the container Publish a container label Sep 23, 2024
@amfage amfage changed the title fix: round footprints to 8 decimal places fix: round footprints to 8 decimal places TDE-1237 Sep 23, 2024
amfage and others added 2 commits September 24, 2024 10:45
Co-Authored-By: Victor Engmark <vengmark@linz.govt.nz>
l0b0
l0b0 previously approved these changes Sep 23, 2024
@amfage amfage marked this pull request as ready for review September 23, 2024 22:52
@amfage amfage requested a review from a team as a code owner September 23, 2024 22:52
@amfage amfage marked this pull request as draft September 26, 2024 22:13
@amfage amfage marked this pull request as ready for review October 2, 2024 22:38
@amfage amfage added this pull request to the merge queue Oct 3, 2024
Merged via the queue into master with commit fbb0930 Oct 3, 2024
2 checks passed
@amfage amfage deleted the fix/truncate-capture-area branch October 3, 2024 21:24
@github-actions github-actions bot mentioned this pull request Oct 3, 2024
github-merge-queue bot pushed a commit that referenced this pull request Oct 7, 2024
🤖 I have created a release *beep* *boop*
---


## [5.0.1](v5.0.0...v5.0.1)
(2024-10-03)


### Bug Fixes

* round footprints to 8 decimal places TDE-1237
([#1088](#1088))
([fbb0930](fbb0930))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
container Publish a container
Development

Successfully merging this pull request may close these issues.

3 participants