Skip to content

Increase chances for building feature generation - #752

Merged
louis-e merged 1 commit into
mainfrom
increase-feature-param
Feb 14, 2026
Merged

louis-e merged 1 commit into
mainfrom
increase-feature-param

Conversation

@louis-e

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

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings February 14, 2026 17:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adjusts residential building window-decoration generation to produce shutters/sills/balconies more frequently, aligning with the goal of increasing visible feature variety in generated buildings.

Changes:

  • Increased shutter placement chance for residential windows.
  • Increased window-sill placement chance per floor/window.
  • Increased balcony placement chance (still gated to the center window column).

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

Comment on lines 1856 to 1859
let shutter_roll =
coord_rng(centre_sum, centre_sum, element.id).random_range(0u32..100);
if shutter_roll < 12 {
if shutter_roll < 25 {
for h in (config.start_y_offset + 1)

Copilot AI Feb 14, 2026

Copy link

Choose a reason for hiding this comment

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

The shutter spawn probability is encoded as a hard-coded threshold (25 out of 100). Since these rates appear to be tuning knobs, consider extracting this into a named constant (or config field) so future balancing doesn’t require hunting for magic numbers and the intent (% chance) is explicit.

Copilot uses AI. Check for mistakes.
Comment thread src/element_processing/buildings.rs
@github-actions

Copy link
Copy Markdown

⏱️ Benchmark run finished in 0m 26s
🧠 Peak memory usage: 1102 MB

📈 Compared against baseline: 30s
🧮 Delta: -4s
🔢 Commit: 4508ea0

🟢 Generation time is unchanged.

📅 Last benchmark: 2026-02-14 17:55:30 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

@louis-e
louis-e merged commit 8dcc8a3 into main Feb 14, 2026
9 checks passed
@louis-e
louis-e deleted the increase-feature-param branch February 14, 2026 18:46
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