Skip to content

More processors#736

Merged
louis-e merged 16 commits intomainfrom
more-processors
Feb 1, 2026
Merged

More processors#736
louis-e merged 16 commits intomainfrom
more-processors

Conversation

@louis-e
Copy link
Owner

@louis-e louis-e commented Feb 1, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 1, 2026 12:16
@github-actions
Copy link

github-actions bot commented Feb 1, 2026

⏱️ Benchmark run finished in 0m 34s
🧠 Peak memory usage: 960 MB

📈 Compared against baseline: 30s
🧮 Delta: 4s
🔢 Commit: d4af5ce

🟢 Generation time is unchanged.

📅 Last benchmark: 2026-02-01 18:44:01 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds support for four new categories of OpenStreetMap elements to be rendered in Minecraft worlds: power infrastructure, historic structures, emergency facilities, and advertising elements. The changes enable the generation of realistic representations of power lines, towers, monuments, fire hydrants, and advertising displays.

Changes:

  • Added support for power infrastructure including transmission towers, utility poles, and power lines with catenary simulation
  • Added historic element processing for memorials, monuments, wayside crosses, and other commemorative structures
  • Added emergency infrastructure support starting with fire hydrants
  • Added advertising structure generation including columns, flags, and illuminated poster boxes

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/retrieve_data.rs Extended Overpass API query to fetch power, historic, emergency, and advertising tagged elements
src/element_processing/mod.rs Declared four new processor modules for the new element categories
src/element_processing/power.rs Implemented power infrastructure generation with realistic towers, poles, and catenary power lines
src/element_processing/historic.rs Implemented historic structure generation including memorials, monuments, and crosses
src/element_processing/emergency.rs Implemented emergency infrastructure starting with fire hydrant generation
src/element_processing/advertising.rs Implemented advertising structure generation with columns, flags, and poster boxes
src/data_processing.rs Integrated new processors into the main processing pipeline for both ways and nodes
src/block_definitions.rs Added 13 new block type definitions including chains with directional variants, end rods, lightning rods, and various decorative blocks

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e louis-e requested a review from Copilot February 1, 2026 14:52
@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 11 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e louis-e requested a review from Copilot February 1, 2026 15:03
@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 11 out of 11 changed files in this pull request and generated 11 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 33 out of 33 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e louis-e requested a review from Copilot February 1, 2026 18:20
@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 33 out of 33 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e louis-e requested a review from Copilot February 1, 2026 18:42
@louis-e
Copy link
Owner Author

louis-e commented Feb 1, 2026

retrigger-benchmark

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 33 out of 33 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@louis-e louis-e merged commit c7e1fec into main Feb 1, 2026
8 checks passed
@louis-e louis-e deleted the more-processors branch February 1, 2026 19:24
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.

2 participants