Skip to content

Conversation

@KumailKamranIS
Copy link
Contributor

Sensitive Credential Checks

  • As the author of these changes, I have checked for any sensitive credentials prior to this review being requested.
  • As a reviewer of these changes, I have checked for any sensitive credentials prior to approving this merge.

Release 0.95.2

Motivation and Context

Description

How Has This Been Tested?

Screenshots (if appropriate):

Checklist:

  • It contains only changes required by issue (does not contain other PR)
  • Includes link to an issue (if apply)
  • I have added tests to cover my changes.
  • I have included my changes in the unreleased section of the changelog.

suarezmashd and others added 30 commits July 10, 2025 12:46
* Multiple changes to accomodate fuel type query

* Added env example

* Final fuel type tweaks
* feature: [DPAV-1492] add migration and endpoint for fetching wind-driven rain data

* feature: [DPAV-1492] dockerise api and run migrations via entrypoint script

* feature: [DPAV-1492] add docstring

* feature: [DPAV-1493] redact password in logs

* feature: [DPAV-1492] project into EPSG:4326
* feature: [DPAV-1488] create icing days endpoint

* feature: [DPAV-1488] add data script to entrypoint script
* feature: [DPAV-1489] add hot summer days api endpoint

* [DPAV-1489] address const sonar

* remove ownership script
Remove bracket from outfile name
…ard-endpoint

Feature/dpav 1456/ward endpoint
…d previously (#80)

* feature: [DPAV-1456] implement script guard if tables have been loaded previously

* feature: [DPAV-1456] tweak main method
* Added functions to download and create the region views within postgis, this will need to be tested within the dev enviroment as constraints on local machine will not run the create views functions, the code is more or less identical to the uk_wards data however the regions are much larger areas so could not be tested on a single region

* Added new migration script for district_borough_unitary_epc, with a change to load_gpkg_to_postgis.py to enable dataviews without joined tables

* typo fixed on first line of 993c0cf8ea04_010_create_uk_district_table_view.py

* table space incorrect name fix!!"

* IRIS - Implement nationwide layer for EPC Ratings

* Update 2215b32f49a9_008_create_uk_ward_table_view.py

Add new data view and  simplify geometry, 0.0001

* Update 8f985d72a651_009_create_uk_region_table_view.py

ST_SIMPLIFY(geometry, 0.0001

* Update 2215b32f49a9_008_create_uk_ward_table_view.py

ST_SIMPLIFY(geometry, 0.0001

* Update 993c0cf8ea04_010_create_uk_district_table_view.py

ST_SIMPLIFY(geometry, 0.0001)

* Update d99eb3e1e4ab_011_create_uk_counties_table_view.py

ST_SIMPLIFY(geometry, 0.0001)

* Default values for DATA_TABLE = "none" and JOIN_TABLE = "none"

* Update 2215b32f49a9_008_create_uk_ward_table_view.py

Spatial index on uk ward view

* fix rouge bracket in migration files

* Update 993c0cf8ea04_010_create_uk_district_table_view.py

ward table referenced in epc table, removed

* Update d99eb3e1e4ab_011_create_uk_counties_table_view.py

uk ward referenced now removed

* Update 993c0cf8ea04_010_create_uk_district_table_view.py

* Update d99eb3e1e4ab_011_create_uk_counties_table_view.py

* add if not exist statements

* more IF EXISTS statements

* revert changes in 001,002,008 and convert 008 to MATERIALIZED VIEW

* revert Makefile after debugging

* index reference changed 011
* Revert "Revert "Feature/dpav 1292/epc endpoints (#81)" (#82)"

This reverts commit 2c99bb0.

* feature: [DPAV-1292] add endpoints for other epc layers
* feature: [DPAV-1481] add roof attributes to structure unit table

* remove invalid syntax
* chore(non-req): updating changelog to prepare for release (#66)

* Update CHANGELOG.md

* feat(dpav-1134): implemented fuel type query into IRIS API (#65)

* Multiple changes to accomodate fuel type query

* Added env example

* Final fuel type tweaks

* Initial implementation of OS NGD Buildings queries

* Updated changelog

* Updated dockerfile

* Update CHANGELOG.md

---------

Co-authored-by: cruddasj <[email protected]>
Co-authored-by: Sam Hill <[email protected]>
Co-authored-by: KumailKamranIS <[email protected]>
… summary endpoints (#88)

* feat(dpav-1684): added os roof data to filterable building and filter summary endpoints

* feat(dpav-1684): updated changelog

* feat(dpav-1684): fixed sonar issues
* feature: [DPAV-1453] add underlying geometry data for Wales

* update crs for geometry column

* feature: [DPAV-1453] resolve comments
#89)

* new postgis fallback option for os ngd buildings data

* Addressed PR comments

* Updated postgres os ngd query

* Addressing sonarcube complaints

* Reverted roof aspect areas back to float,  improved os ngd mapper logic, handled postgis solar panel processing
* New pg query for os ngd buildings

* Updating core api components for roof shape

* Addressing sonarcube complaints
jsmith-informed and others added 23 commits November 7, 2025 11:41
- Reuse db engine
- Add DB_QUERY_TIMEOUT setting, 30s default
- Add execute_with_timeout helper to use higher timeouts
- Increased timeout on filter-summary endpoint (slow if done on large area)
- Fix outdated tests to match actual implementation
- Add unit tests to CI and add makefile command to match PR template
…iew (#142)

This view pre-calculates sums and counts by snapshot date, geography and
property type for faster querying on timeline charts.
- Add area_level and area_names params to dashboard queries
- Add endpoints to get area names, county, ward etc
Using the pre-calculated aggregations where possible, also updated
the average sap endpoint to do the same.
Isolated in the query layer to limit changes, we may rebuild the
materialized views to perform this transformation at that point.
* feat(DPAV-1779): add epc by area dashboard endpoint

Update the existing epc count query to make use of the aggregates
table for better performance as well.

* feat(DPAV-1954): add dashboard epc-ratings-by-feature endpoint

* feat(OSPO): synchronise OSPO workflows

---------

Co-authored-by: jsmith-informed <[email protected]>
This was trying to use the transaction again when it is in a failed state
by issueing statement_timeout SET in finally block.

Instead use LOCAL so it only applies to the transaction, and reset it
after the query on success only. If it timesout or fails it will be
reset when the transaction ends anyway so no need for us to set it back.

Also increased the default timeout and removed redundant fallback value
that was added for testing config issue previously.
)

* bugfix(dpav-1926): changed the logic for geometric matching to contains for the analytics view

* feat(OSPO): synchronise OSPO workflows
…ipt (#157)

* bugfix(dpav-1926): corrected the join logic in the sync region fk script

* bugfix(dpav-1926): sort out issue flagged by OSS check
* feat(dpav-2060): reverted db query timeout back to 30 seconds

* feat(dpav-2060): changed timeout to 29 seconds
* chore(non-req): updating changelog to prepare for release (#66)

* Update CHANGELOG.md

* feat(dpav-1134): implemented fuel type query into IRIS API (#65)

* Multiple changes to accomodate fuel type query

* Added env example

* Final fuel type tweaks

* chore(non-req): fixes after sorting out conflicts

---------

Co-authored-by: cruddasj <[email protected]>
Co-authored-by: Sam Hill <[email protected]>
Co-authored-by: suarezmashd <[email protected]>
…ding feature charts (#161)

* feature(non-req): updated indices to optimise for performance of building feature charts

* chore(non-req): updated changelog
@github-actions github-actions bot added documentation Improvements or additions to documentation release actions labels Jan 15, 2026
@github-actions
Copy link

✅ OSS Checks Passed

All tracked OSS checks passed in this run.

📊 Total Files 🟢 Passed 🔴 Failed 🧮 Score
13 13 0 100%

Results from commit e0859da, view the full job summary↗️ for detailed results.

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
4 Security Hotspots
E Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

actions documentation Improvements or additions to documentation release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants