-
-
Notifications
You must be signed in to change notification settings - Fork 700
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
Show raster row/column of clicked point in identify results #8764
Conversation
Interesting remark. The identify tool description is quite vector centered right now despite it can be used also with raster, mesh or point cloud. We could move/create a section for results dialog (and format specifics) to each data type chapter, leaving the common how-to parts and options description in the general tools chapter. Just a rough idea (unfortunately not resulting in a single line PR 😄) and I'm open to all options... |
I have added the "Introduction to Raster Data" chapter and provided details about the Identify tool. I will create new PRs for point cloud, mesh, and vector data. |
Supported formats include GeoTIFF, Erdas Imagine, ArcInfo ASCII Grid, and many others. | ||
See more at :ref:`opening_data`. | ||
|
||
**Worth knowing** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's put emphasis on this subject
**Worth knowing** | |
Identifying raster cells | |
======================== |
and I wonder now if this should be here or after the "properties" block
@@ -1121,6 +1121,9 @@ default it will display the following information: | |||
closest vertex (and ``Z``/``M`` if applicable) | |||
* if you click on a curved segment, | |||
the radius of that section is also displayed. | |||
* if you use :guilabel:`Identify features` on a raster layer, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
refs comments on #8864
* Select the raster layer in the Layers panel. | ||
* Click on the :guilabel:`Identify features` tool in the toolbar or press :kbd:`Ctrl+Shift+I`. | ||
* Click on the point in the raster layer that you want to identify. | ||
|
||
The Identify Results panel will open and display information about the clicked point. | ||
For raster layers, this includes the row and column of the clicked point. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Select the raster layer in the Layers panel. | |
* Click on the :guilabel:`Identify features` tool in the toolbar or press :kbd:`Ctrl+Shift+I`. | |
* Click on the point in the raster layer that you want to identify. | |
The Identify Results panel will open and display information about the clicked point. | |
For raster layers, this includes the row and column of the clicked point. | |
#. Select the raster layer in the Layers panel. | |
#. Click on the :guilabel:`Identify features` tool in the toolbar or press :kbd:`Ctrl+Shift+I`. | |
#. Click on the point in the raster layer that you want to identify. | |
The Identify Results panel will open and display information about the clicked point. | |
For raster layers, this includes the row and column of the clicked point. |
and let's mention all particular information people should expect here
@@ -1,6 +1,33 @@ | |||
.. index:: Raster, Layer properties | |||
.. _raster_properties_dialog: | |||
|
|||
*************************** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is weird that the build passes with two identical topmost underlines (i.e. page title) in the same file. Anyway the generated HTML is worth a look and you might need to review the sectioning here. E.g. we usually have a table of content displayed just below the page title; here it might be under raster properties section, far below in the page.
@@ -1,9 +1,43 @@ | |||
.. index:: Raster, Layer properties | |||
|
|||
************************** | |||
Intoduction to Raster Data |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intoduction to Raster Data | |
Introduction to Raster Data |
but if this is the top title I don't know if "introduction" is the most suitable for the addressed topics. Introduction is just for the first two paragraphs. that said, I have no suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree... Is it possible to retain the text but remove the sectioning (title), if that makes sense? I was considering removing it entirely...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assuming I understood correctly, I'd find weirder to have a new chapter/page that starts with no title.
Raster data can be stored in serval supported formats, including: | ||
|
||
* GeoTIFF | ||
* ERDAS Imagine | ||
* ArcInfo ASCII Grid | ||
* PostGIS Raster and others. | ||
|
||
See more at :ref:`opening_data`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Raster data can be stored in serval supported formats, including: | |
* GeoTIFF | |
* ERDAS Imagine | |
* ArcInfo ASCII Grid | |
* PostGIS Raster and others. | |
See more at :ref:`opening_data`. | |
Raster data can be stored in several supported formats, including GeoTIFF, | |
ERDAS Imagine, ArcInfo ASCII Grid, PostGIS Raster and others... | |
See more at :ref:`opening_data`. |
Raster Properties Dialog | ||
************************ | ||
======================== | ||
|
||
.. only:: html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reading my previous comment it was probably not clear but the TOC should be moved up, under the top title, to catch all sections of the page. and not stay in the middle of the page.
[Backport release_3.34] Show raster row/column of clicked point in identify results (backports #8764)
Fixes #8500
I believe that such 'small details' should be mentioned in the 'Working with Raster' chapter, but I'm not sure how to incorporate them into that chapter. I would like to hear suggestions and opinions on this.