Skip to content

Conversation

@ccbblin
Copy link
Contributor

@ccbblin ccbblin commented Jun 13, 2025

Motivation

Tile size report is large, we will have two reports for Webmecator and NZTM.

Modifications

collapse report

Verification

@ccbblin ccbblin force-pushed the fix/collapse-analysis-report branch from 4657882 to 2c03625 Compare June 13, 2025 03:59
@ccbblin ccbblin added the container build pull request container for testing label Jun 13, 2025
@ccbblin ccbblin marked this pull request as ready for review June 15, 2025 21:30
@ccbblin ccbblin requested a review from a team as a code owner June 15, 2025 21:30
@Wentao-Kuang
Copy link
Contributor

I got this error from my previous changes. Could you please fix it together in here. I think we need check exists before fsa.read for the report.mb.

@ccbblin
Copy link
Contributor Author

ccbblin commented Jun 16, 2025

I got this error from my previous changes. Could you please fix it together in here. I think we need check exists before fsa.read for the report.mb.

ah you fixed this already :)

if (await fsa.exists(fsa.toUrl(reportPath))) {
We need to do a new release

@ccbblin ccbblin added this pull request to the merge queue Jun 17, 2025
Merged via the queue into master with commit 19a4d85 Jun 17, 2025
14 checks passed
@ccbblin ccbblin deleted the fix/collapse-analysis-report branch June 17, 2025 02:07
@ccbblin ccbblin mentioned this pull request Jun 17, 2025
github-merge-queue bot pushed a commit that referenced this pull request Jun 17, 2025
### Motivation

# [8.3.0](v8.2.0...v8.3.0)
(2025-06-17)


### Bug Fixes

* **cli-vector:** collapse large analysis report in pr comment
([#3457](#3457))
([19a4d85](19a4d85))
* **infra:** increase lambda tiler memory to 3072MB
([#3459](#3459))
([786bb54](786bb54))
* **landing:** Fix debug page automatically hide topographic-v2. BM-1302
([#3456](#3456))
([53134c0](53134c0))


### Features

* **cli-config:** Generate topographic previews with the v2 in the url.
BM-1203 ([#3455](#3455))
([1a2f220](1a2f220))
* **cli-vector:** Support NZTM mbtiles creation.BM-1300
([#3452](#3452))
([f601a73](f601a73))
* **landing:** Support NZTM vector Links in the menu page. BM-1301
([#3458](#3458))
([a57059e](a57059e))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

container build pull request container for testing

Development

Successfully merging this pull request may close these issues.

3 participants