Skip to content

Dev/ai summary dev#57

Draft
schmidtnz wants to merge 16 commits into
masterfrom
dev/ai-summary-dev
Draft

Dev/ai summary dev#57
schmidtnz wants to merge 16 commits into
masterfrom
dev/ai-summary-dev

Conversation

@schmidtnz

Copy link
Copy Markdown
Contributor

Motivation

Modifications

Verification

kbaker-linz and others added 10 commits July 1, 2026 10:52
Trying to get this project file up to date before making other changes.
Removed the last layer reference to nz_topo50*.
Added the all of government logo.
Added the correct information for the text colour on carto text - process_blue.
removed the gridded map service that was causing an issue
removed relief service
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🗺️ QGIS project change summary

map-series/nztopo50/nztopo50.qgs

The project was reworked for an overview/index map setup: overview sheet/coastline layers were added, some reference layers were removed, and a few layers now only draw within defined scale ranges.

  • Layers

    • Removed Cartographic Text (the filtered nz_topo50_carto_text.parquet layer with text_string is not null).
    • Replaced nz_topo50_map_sheet with three layers from ./nztopo50_map_sheet.parquet:
      • nztopo50_map_sheet - overview with subset sheet_code = 'BW15'
      • nztopo50_map_sheet - index current with subset sheet_code = 'BW15'
      • nztopo50_map_sheet - index all with no subset
    • Added coastline - overview from ./coastline.parquet.
    • Removed raster background/reference layers Topo Maps and Hillshade Igor Color ramp.
    • Reordered the Structure map group to appear below Transport map in the layer tree.
  • Scale

    • Enabled scale-based visibility for DMS grid and Grid lines, both with minScale="300000".
    • Enabled scale-based visibility for bridge_line copy with minScale="100000".
  • Project

    • Updated custom layer order to remove the deleted Cartographic Text, nz_topo50_map_sheet, Topo Maps, and Hillshade Igor Color ramp entries, and add the new overview/index layers.
  • Churn

    • QGIS build version changed (4.1.0-Master4.0.3-Norrköping); this is save/build churn, not a project edit.

Generated by gpt-5.4 via the GitHub Copilot CLI from a churn-free diff - verify against the file diff.

schmidtnz added a commit that referenced this pull request Jul 9, 2026
This pull request improves the QGIS project diff summarization workflow
by enhancing how diffs are processed, summarized, and presented. The
changes focus on making the AI-generated summaries more accurate,
concise, and robust against irrelevant "churn" (noise from QGIS/Qt build
differences), while also refining the workflow's reliability and
maintainability.

**Workflow and AI summary improvements:**

- The workflow now inlines the diff directly into the Copilot prompt
(instead of reading from a file), with a character cap to avoid argument
length errors, and clearly displays the diff size in the logs.
- The AI prompt is now sourced from a dedicated markdown file
(`qgs-review-prompt.md`), which provides more precise and maintainable
instructions for generating summaries.
[[1]](diffhunk://#diff-20eeda2d3ff0bcbaba1993497a03a3a9bc6d2c75e889c49eb34b11c6afee95ccR62-R96)
[[2]](diffhunk://#diff-061ce3dabd46e96c4a36f407662160cd8c3aa58b87d1917be3d56a2ef3530190R1-R39)
- The Copilot model is updated from `gpt-5-mini` to `gpt-5.4` for
improved summary quality.

**Churn reduction and diff simplification:**

- The diff simplification script (`qgs-diff.py`) now strips trailing
QFont descriptor fields (`,,0,0`) that are added by newer QGIS/Qt
builds, further reducing irrelevant churn in diffs.
[[1]](diffhunk://#diff-84448358df0fb25639c99f298878713e27ec4fd711005c24ca52f5f5c054cfe0L26-R28)
[[2]](diffhunk://#diff-84448358df0fb25639c99f298878713e27ec4fd711005c24ca52f5f5c054cfe0R85-R93)

**Prompt and summary formatting enhancements:**

- The new markdown prompt (`scripts/assets/qgs-review-prompt.md`) gives
detailed, structured instructions for distinguishing real changes from
build churn, formatting summaries, and collapsing repetitive noise into
concise notes.

These changes collectively make the QGIS project diff summaries more
accurate, scannable, and focused on substantive changes, while
minimizing false positives from build artifacts.


### Validation
Test branch / comment here:
#57 (comment)
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