MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002778VCMIOtherpublic2017-08-27 02:152017-08-29 07:02
Reporterhkoehler 
Assigned To 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusnewResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0002778: INDEPENDENT_MIN and INDEPENDENT_MAX logics are a mess
DescriptionCurrently we allow mixing of INDEPENDENT_MIN, INDEPENDENT_MAX and other value types. The result is a mess (see BonusList::totalValue).
Additional InformationShould log error when encountering combination of different value types (maybe also change logic). Then add new value types LOWER_BOUND and UPPER_BOUND.
TagsNo tags attached.
Attached Files

- Relationships

-  Notes
(0007242)
Warmonger (administrator)
2017-08-29 06:40

But what you propose? LOWER_BOUND and UPPER_BOUND would be just renamed INDEPENDENT_MIn and INDEPENDENT_MAX.

Current formula takes all possibilities into consideration, as it's descibed on Wiki:
http://wiki.vcmi.eu/index.php?title=List_of_bonus_value_types [^]
(0007243)
hkoehler (developer)
2017-08-29 06:52
edited on: 2017-08-29 07:02

The way the wiki describes it would be sensible, however:
* Currently INDEPENDENT_MIN is the maximum value taken and INDEPENDENT_MAX the minimum value, which is fairly confusing UNLESS you never mix them with other bonuses (as I was told is current practice). Here renaming would help stave off confusion (we might have to introduce a mapping to deal with usage in existing mods).
* The actual implementation does not follow the wiki. There's an assertion that INDEPENDENT_MIN < INDEPENDENT_MAX which should be the other way around (someone got confused already), and if no non-independent bonuses are present (i.e., the "normal" case) then it's different from having a non-independent bonus of 0.


- Issue History
Date Modified Username Field Change
2017-08-27 02:15 hkoehler New Issue
2017-08-29 06:40 Warmonger Note Added: 0007242
2017-08-29 06:52 hkoehler Note Added: 0007243
2017-08-29 06:53 hkoehler Note Edited: 0007243 View Revisions
2017-08-29 06:53 hkoehler Note Edited: 0007243 View Revisions
2017-08-29 07:02 hkoehler Note Edited: 0007243 View Revisions

Site | Forums | Wiki | Slack | GitHub


Copyright © 2000 - 2020 MantisBT Team
Hosting provided by DigitalOcean