Skip to content

Conversation

@efrec
Copy link
Collaborator

@efrec efrec commented Nov 19, 2025

Work done

  • Adds modoptions wreck_metal_ratio and heap_metal_ratio for testing reclaim amounts. Usage is !bset wreck_metal_ratio 0.5 for 50% of unit metal value.
  • Does not rebalance build costs of eating/rezzing so the min/max modoption values are bounded semi sensibly.
  • Adjusted the reclaim highlights widget slightly (square of the difference) to filter out minimum feature metal/energy based on the adjusted value.

Values for wreck_metal_ratio should be between 0.2 and 0.8 (default 0.6) (ish) given current game design. I don't think I've seen any pronouncements on heaps.

This was requested for balance testing by coordinators so should be considered temporary. I don't mind adding an end date to the modoption (since that's been discussed as a feature to implement) or waiting on that feature to add this.

This could be considered a cheat option, especially when increasing metal reclaim from Raptors, but I didn't see the point in moving it to the cheats section, personally.

efrec added 2 commits November 19, 2025 16:55
This isn't the most neat code since it keeps the wreck metal% default value in two places. I didn't want to move that value default entirely into the modoption logic, though.
@github-actions
Copy link
Contributor

github-actions bot commented Nov 19, 2025

Test Results

14 tests  ±0   7 ✅ ±0   9s ⏱️ ±0s
 1 suites ±0   7 💤 ±0 
 1 files   ±0   0 ❌ ±0 

Results for commit 6072e51. ± Comparison against base commit ea60385.

♻️ This comment has been updated with latest results.

@efrec
Copy link
Collaborator Author

efrec commented Nov 19, 2025

re: not respecting modoption limits, here's an 80x metal modifier

{2A66F582-D28B-4B79-B177-66CD38CE5BD8} {0A4850F2-BC81-4196-907C-D1CC806D296D}

*edit: I've capped the multipliers to be non-cheaty.

@TheChronographer
Copy link
Collaborator

TheChronographer commented Nov 20, 2025

The other way to do this, to balance test specific units, could be adding a custom param of 'non uniform heap/wreck values' and then adding an exemption to the alldefs_post like commanders have.

Could be useful to allow 'raiding units' (e.g. blitz) donate less metal as a balance testing tool.

@efrec
Copy link
Collaborator Author

efrec commented Nov 20, 2025

K this is much-simplified.

A tweaks approach will be a separate PR. I think we want to keep using these ratios and not allow them to become too arbitrary. But still there are things the coordinators will want to test per-unit.

Copy link
Contributor

@keithharvey keithharvey left a comment

Choose a reason for hiding this comment

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

LGTM

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.

4 participants