feat: uses ACAR to resolve refused scenarios. #5792
Conversation
| public Color getBelowContractMinimumForeground() { | ||
| return new Color(userPreferences.node(MHQConstants.DISPLAY_NODE) | ||
| .getInt(MHQConstants.BELOW_CONTRACT_MINIMUM_FOREGROUND, Color.RED.getRGB())); | ||
| .getInt(MHQConstants.BELOW_CONTRACT_MINIMUM_FOREGROUND, CRIMSON_RED.getRGB())); |
There was a problem hiding this comment.
If we're updating this, it would be better to use the negative or warning event color, so users can configure it.
There was a problem hiding this comment.
maybe, but I dont know what you are referencing here
| if (stub) { | ||
| if (getCampaignOptions().isUseStratCon() && (scenario instanceof AtBDynamicScenario atBDynamicScenario)) { | ||
|
|
||
| // if (processIgnoredScenario(atBDynamicScenario, contract.getStratconCampaignState())) { |
There was a problem hiding this comment.
If we're confident we're making this move, we probably want to remove the code rather than commenting it out.
| if (scenario.isTurningPoint()) { | ||
| campaignState.updateVictoryPoints(-1); | ||
| } | ||
| // this is already processed in the simulateTheResultsOfIgnoredScenario |
There was a problem hiding this comment.
Here, too. If we're removing this functionality let's remove the dead code instead of just commenting it out.
It would run ACAR for each refused scenario, I would still add a campaign option for it to be activated (I guess people could dislike the fact that the auto resolve is a homebrew system). For some reason it went up as a normal PR and not as a draft, its not by any means ready, so there is some things missing that I want to add before calling that a feature. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5792 +/- ##
=======================================
Coverage ? 10.29%
Complexity ? 6124
=======================================
Files ? 1038
Lines ? 139120
Branches ? 20600
=======================================
Hits ? 14327
Misses ? 123398
Partials ? 1395 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I will leave this PR open as a Draft. It is small enough that can be picked up again in the future, however it has too many small problems, and I don't know how it can "actually" give anything to us unless we have persistent OpFor. Otherwise it will be as random as just disappearing with a scenario by rolling a dice |
Experiment - using ACAR to resolve refused scenarios.
If there is team 1 in the scenario in the form of attached allies, it will run a game and see what happens, otherwise it follows the normal ways.