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

Suggestion: tweak text for proj:transform to warn off common pitfalls #9

Closed
Kirill888 opened this issue Feb 23, 2022 · 1 comment · Fixed by #10
Closed

Suggestion: tweak text for proj:transform to warn off common pitfalls #9

Kirill888 opened this issue Feb 23, 2022 · 1 comment · Fixed by #10

Comments

@Kirill888
Copy link
Contributor

projection/README.md

Lines 101 to 109 in 7f1d1c9

#### proj:transform
Linear mapping from pixel coordinate space (Pixel, Line) to projection coordinate space (Xp, Yp). It is
a `3x3` matrix stored as a flat array of 9 elements in row major order. Since the last row is always `0,0,1` it can be omitted,
in which case only 6 elements are recorded. This mapping can be obtained from
GDAL([`GetGeoTransform`](https://gdal.org/api/gdaldataset_cpp.html#_CPPv4N11GDALDataset15GetGeoTransformEPd)) or the Rasterio
([`Transform`](https://rasterio.readthedocs.io/en/stable/api/rasterio.io.html#rasterio.io.BufferedDatasetWriter.transform)).
To get it on the command line you can use the [Rasterio CLI](https://rasterio.readthedocs.io/en/latest/cli.html) with the
[info](https://rasterio.readthedocs.io/en/latest/cli.html#info) command: `$ rio info`.

Description above mentions GetGeoTransform and rio info. These two produce different formats on output, and while this extension is compatible with rio info, data from GetGeoTransform needs to be re-arranged. I think it's easy enough to assume that you can just stick output of GetGeoTransform into the stac document, but this is not the case.

@hobu
Copy link

hobu commented Feb 23, 2022

Make a PR that updates the language and it is quite likely to be merged.

Kirill888 added a commit to Kirill888/projection that referenced this issue Feb 23, 2022
matthewhanson added a commit that referenced this issue Apr 11, 2022
Add clarification about `proj:transform` (#9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants