Skip to content

Fix Monte Carlo reliability sampling to stay within configured bounds#1

Draft
nicklasorte wants to merge 1 commit intomainfrom
codex/investigate-aggregate-interference-threshold-issue
Draft

Fix Monte Carlo reliability sampling to stay within configured bounds#1
nicklasorte wants to merge 1 commit intomainfrom
codex/investigate-aggregate-interference-threshold-issue

Conversation

@nicklasorte
Copy link
Copy Markdown
Owner

Motivation

  • The prior random-scaling expression could generate values below min(reliability), which can distort EIRP interpolation and create mismatch between move-list sizing and aggregate-check results.

Description

  • Replace the old scaling with explicit bounds by introducing rel_min = min(reliability); rel_max = max(reliability); rand_numbers = rand(num_rows,1)*(rel_max-rel_min)+rel_min; in monte_carlo_super_bs_eirp_dist_rev3.m so draws are constrained to [rel_min, rel_max].

Testing

  • Ran nl -ba monte_carlo_super_bs_eirp_dist_rev3.m | sed -n '1,80p' to inspect the file and ran rg -n "rand(num_rows,1)*(max(reliability)-min(reliability)+min(reliability))" monte_carlo_super_bs_eirp_dist_rev3.m to confirm the old expression is absent, and both checks succeeded.

Codex Task

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant