Skip to content

docs: drop removed --numerical-guard-mm flag and field from balancing region example - #1253

Open
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/fix-docs/docs-drop-removed-numerical-guard-mm-flag-and-fiel-589e69
Open

detail-app[bot] wants to merge 1 commit into
mainfrom
detail/fix-docs/docs-drop-removed-numerical-guard-mm-flag-and-fiel-589e69

Conversation

@detail-app

@detail-app detail-app Bot commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

The board-array-balancing-region example doc still listed numerical_guard_mm as a
BalancingRegionOptions field, documented a --numerical-guard-mm CLI flag (default
0.025), and described the construction guard e as the fixed value 0.025. After the
accuracy-budget refactor, that field/flag no longer exists on either the options
struct or the example's Args; e is now derived as half the panel accuracy budget
(default 0.005 mm).

Running the example with the documented --numerical-guard-mm flag would fail with an
"unknown argument" error, and copying the BalancingRegionOptions struct snippet would
fail to compile against current pcb-ir.

Introduced by commit 10ec137 (@akhilles, #1171)


Doc Drift PRs can be configured here.


Note

Cursor Bugbot is generating a summary for commit 71cd631. Configure here.

@detail-app
detail-app Bot requested a review from akhilles September 8, 2026 06:05

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔍 Devin Review: 2 flags

Not posted on this PR by your GitHub settings — view them in Devin Review. (Configure)

Devin Review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Approved: this is a small docs-only fixup removing a stale flag/field from one example. Cursor Bugbot and Cursor Security Agent both completed successfully with no findings that need human review, and no reviewers were assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

This branch has not been deployed

No deployments
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