CI test: contour labeling fixes#4
Open
Geojim wants to merge 3 commits into
Open
Conversation
Geojim
force-pushed
the
raster-contour-labeling
branch
2 times, most recently
from
4b21c86 to
432b6e1
Compare
Introduces on-the-fly contour line labels for raster DEM layers using GDAL's contour generation API. Contour geometries are generated at render time and registered with the PAL labeling engine for placement along the lines. New classes: - QgsRasterContourLabelProvider: generates contour lines from raster data and registers them as label features - QgsRasterLayerContourLabeling: configuration class with text format, numeric format, thinning, and scale-based visibility settings The label provider reads band, contour interval, index interval, and downscale factor from QgsRasterContourRenderer (Symbology tab) to avoid settings duplication. Sketched-With: Claude <noreply@anthropic.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Fix SIP include ordering (alphabetical) - Add Doxygen comments to all public methods - Rename _contourLabelWriter to avoid reserved identifier (clang-tidy) - Remove extra blank lines (pre-commit) - Update tests to match removed band/interval/downscale accessors Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Geojim
force-pushed
the
raster-contour-labeling
branch
from
432b6e1 to
1e1bcdf
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Testing CI before upstream resubmission