[Babak (babaknaderi)]

Warn when bonus thresholds fall outside the recommended range or cannot be reached because of study and worker HIT limits.

[Copilot]

🟡 Changes recommended

The validator raises KeyError when a previously optional worker HIT limit is omitted, preventing study generation.

1 open issue

• Avoid KeyError when allowed_max_hit_in_project is omitted · New

What changed in this PR

This PR adds bonus-setting warnings to study generation, checking recommended thresholds and whether study or worker HIT limits make the quantity bonus unreachable.

Changes:

• Extracts bonus validation into a reusable warning helper.
• Reports inconsistent HIT limits and unreachable bonus thresholds.
• Adds parameterized warning tests and updates an existing test configuration.

File	Description
tests/​test_master_script.py	Adds warning cases and supplies a worker HIT limit in an existing fixture.
src/​master_script.py	Adds bonus-setting validation and prints its warnings during study generation.

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

ACR validation uses a higher default worker limit than result analysis, allowing unreachable bonuses to pass without warnings.

1 open issue

• Use effective HIT acceptance limit for ACR fallback validation · New

1 resolved since last review

• Avoid KeyError when allowed_max_hit_in_project is omitted

_{Review effort: Balanced}