Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

research.json: Add support for new "calculationMode" option, "Improved" calculation mode #3796

Merged

Conversation

past-due
Copy link
Member

@past-due past-due commented May 7, 2024

The default "calculationMode" is "compat", and functions as many previous versions for many years have. This mode can accumulate noticeable error - especially if repeatedly upgrading small values by smaller percentages (commonly impacted: armour, thermal).

The new opt-in "calculationMode" is "improved". This handles calculating upgrades in a way that significantly reduces accumulated errors from multiple applied research upgrades to a component's value(s).

To opt-in to the new mode, add a new "config" dict to the top-level research.json object, specifying the desired "calculationMode".

Example:

{
  "_config_": {
    "calculationMode": "improved"
  },
  ...
}

@past-due past-due force-pushed the 2024_04_research_upgrade_calculations_1 branch from 18fa68c to 79dc8b4 Compare May 7, 2024 20:48
The default `"calculationMode"` is `"compat"`, and functions as many previous versions for many years have. This mode can accumulate noticeable error - especially if repeatedly upgrading small values by smaller percentages (commonly impacted: armour, thermal).

The new opt-in `"calculationMode"` is `"improved"`. This handles calculating upgrades in a way that significantly reduces accumulated errors from multiple applied research upgrades to a component's value(s).

To opt-in to the new mode, add a new "_config_" dict to the top-level research.json object, specifying the desired "calculationMode".

Example:
```json
{
  "_config_": {
    "calculationMode": "improved"
  },
  ...
}
```
@past-due past-due force-pushed the 2024_04_research_upgrade_calculations_1 branch from 79dc8b4 to 32d8fcf Compare May 7, 2024 20:50
@past-due past-due merged commit 1c20240 into Warzone2100:master May 7, 2024
37 checks passed
@past-due past-due deleted the 2024_04_research_upgrade_calculations_1 branch May 7, 2024 22:59
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.

2 participants