Skip to content

Stream Overture building stats by batch - #356

Open
mividtim wants to merge 8 commits into
larsiusprime:masterfrom
fabrica-land:strudel/eng-3033-overture-streaming-upstream
Open

Stream Overture building stats by batch#356
mividtim wants to merge 8 commits into
larsiusprime:masterfrom
fabrica-land:strudel/eng-3033-overture-streaming-upstream

Conversation

@mividtim

@mividtim mividtim commented Jun 9, 2026

Copy link
Copy Markdown

Summary:

  • add a streaming Overture building-batch path for parcel building stats
  • use the streaming path from Overture enrichment so dense bboxes do not materialize every footprint at once
  • keep get_buildings compatible for existing callers
  • add old-vs-streaming equivalence tests for footprint area, max height, max floors, empty inputs, and no-height inputs

Verification:

  • focused Overture pytest in the AVM worker venv: 6 passed
  • git diff --check: clean

Note: a broader local tests/test_data.py run is blocked in the reused AVM worker venv by missing ngboost; the focused Overture tests run in the same geospatial dependency stack used by the AVM worker.

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution.
Please sign our CLA at the following link:
Click here to sign the CLA.

No action is required from you in this PR thread. Once you have signed the CLA externally, a maintainer will verify your signature and record it here on your behalf by commenting:


I affirm that this contributor has signed the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@fabrica-cc-engineering-agent
fabrica-cc-engineering-agent Bot deleted the strudel/eng-3033-overture-streaming-upstream branch June 23, 2026 14:08
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.

1 participant