-
-
Notifications
You must be signed in to change notification settings - Fork 220
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
Add raster zxy function doc #937
base: main
Are you sure you want to change the base?
Conversation
If it is a user-defined function, I don't think we should actually add any tiff-parsing/generation to the integration testing. Unless we can make Martin recognize image tables and automatically create functions for them, I do not think we should add anything beyond a simple test without raster support. Testing strategy
|
ae352e6
to
5774b36
Compare
Could you reduce the size of the tiff please? A 0.7MB file is a bit large for a test like this. Also, do you think we should also include some PNG and JPG samples into PG serving? Lastly - while this test is good to have, the main use of it is to have detailed description in the doc book - please add some info there, either as a small paragraph, or maybe even a dedicated page with in-depth examples. |
Hi @nyurik Have a good day!
Any guidance and help would be really appreciated. : ) |
I do think we should also include really simple and tiny tests if we guaranteed we could serve raster function. |
@sharkAndshark if your goal is to simply document serving raster tiles, please re-read #937 (comment) -- you do not need anything TIFF related, or anything related to configuring postgis code. Instead, much easier is to add PNG or JPG files directly into a Postgres table as a bytea value. |
@sharkAndshark where did you find this image? I think it might be ok for documentation purposes to have an image-serving setup, and 0.7MB might be ok, but we would need to add proper copyright to it. Best is to find NASA-generated images (they are public domain), or some image from Wikipedia Commons (they have open licenses). |
Thx. I can't remember the website. I will try to find another small and proper licensed image. Try NASA first. |
The main goal of this PR is try to add doc about how to support raster in PostGIS with user defined function.
The test in this PR might be remove after it works before merged.