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

Update gdal_contour.rst #11710

Closed
wants to merge 1 commit into from
Closed

Update gdal_contour.rst #11710

wants to merge 1 commit into from

Conversation

jidanni
Copy link
Contributor

@jidanni jidanni commented Jan 24, 2025

See https://en.wikipedia.org/wiki/Digital_elevation_model#Types

Also the user worries that their XYZ CSV file might not be an official DEM, though it will work just fine.

See https://en.wikipedia.org/wiki/Digital_elevation_model#Types

Also the user worries that their XYZ CSV file might not be an official DEM, though it will work just fine.
@jratike80
Copy link
Collaborator

What if another user, maybe an OGC member, worries that they cannot create contours from their temperature data? Only "Regularly gridded coverage data file" could make them satisfied (maybe, needs to be discussed in the SWG). They might also argue that XYZ CSV is not raster, it is point data that can expressed as a raster, but only in a special case when it makes up a regular grid.

I do not believe that the meaning of DEM has ever misunderstood, not even by any OGC member. The word "raster" appears there already raster elevation model (DEM). But if you think that raster elevation model (e.g., a raster DEM) is needed to make it clear, it is OK for me.

@rouault
Copy link
Member

rouault commented Jan 24, 2025

I believe the current description is enough clear / unambiguous, so in the interest of keeping text shorter when possible, I'll close without merging

@rouault rouault closed this Jan 24, 2025
@jidanni
Copy link
Contributor Author

jidanni commented Jan 24, 2025

I mean it

The gdal_contour generates a vector contour file from the input raster elevation model (DEM).

looks like it should say

input digital elevation model (DEM).

or

input raster elevation model (REM).

It looks rather asleep at the wheel.

@jratike80
Copy link
Collaborator

Or just "The gdal_contour generates a vector contour file from the input raster."
The utility does not know if the input is DEM or DSM, or some temperature, pressure etc. data. That we deal with digital data is obvious for most users, I believe.
In https://gdal.org/en/stable/programs/gdaldem.html "elevation raster" and "DEM raster" are used. Non-uniform, but still correctly understood.
REM? It makes me do rapid eye movements, never heard in this context.

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 this pull request may close these issues.

3 participants