#3510 closed defect (fixed)
[PATCH] Adjust HP for Celtic and Spartan female citizens
Reported by: | Palaxin | Owned by: | scythetwirler |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 20 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description (last modified by )
In r16757 HP for female citizens were halfed. Since Celtic and Spartan females have hardcoded HP in their templates, they have become quite overpowered compared to "normal" females.
A simple fix is to adjust the HP in the civ templates. The float HP of some Celtic females are not rounded from 37.5 to 38, so at full HP 38/37.5 is displayed. Perhaps a new ticket for this problem is needed?
The other solution is to add two autoresearch technologies which multiply the standard female HP by the desired factor. So you have only to adjust the HP once and they are scaled automatically for Celts and Spartans. Interestingly the display problem did not occur here. The technology names and descriptions could be improved, I just used a shortened version of the female citizen template history. I also included the bonus attack damage for Celtic females in the new technology.
Attachments (5)
Change History (17)
by , 9 years ago
Attachment: | female_simple.patch added |
---|
by , 9 years ago
Attachment: | defect_hp_display.jpg added |
---|
by , 9 years ago
Attachment: | correct_hp_display.jpg added |
---|
by , 9 years ago
Attachment: | female_technologies.patch added |
---|
comment:1 by , 9 years ago
Description: | modified (diff) |
---|
comment:2 by , 9 years ago
Description: | modified (diff) |
---|
comment:3 by , 9 years ago
Milestone: | Backlog → Alpha 20 |
---|---|
Priority: | Must Have → Should Have |
comment:4 by , 9 years ago
comment:7 by , 9 years ago
The technologies have one major disadvantage: If you are under control of female citizens of a different civ (e.g. a special scenario or capturing) they will get or lose the bonus though in my opinion they shouldn't. So I'd really prefer to define the HP in the templates as it is done now --> simple patch.
Thinking further this problem probably also affects other technologies: hellenic structures have autoresearched 10% bonus HP, but what if you captured them with a non-hellenic civ or even brits/gauls which have -20% HP for structures?
by , 8 years ago
Attachment: | femal_rel_templates_not_working.patch added |
---|
comment:8 by , 8 years ago
I tried to make another patch version with the relative templates implemented in r17386. Sadly it does not work that way. I've no clue what is wrong...
comment:9 by , 8 years ago
Seems to work for me. There hasn't been an autobuild since r17386 though, so you'd need to compile it yourself to see the effects.
comment:10 by , 8 years ago
Ahhh that makes sense :) In fact I didn't compile it myself. Thank you. I will wait for the next autobuild to test it though
I just realized that the technology "health_females_01.json" also gives 50% bonus HP. Since "multiply" affects only the base HP value we get different HP values for the following two cases (let's take Spartan females with 80% increased base HP as example):
So it seems only the simple fix won't change the current relative difference of max HP between the civs. A nice feature would be to implement an additional "multiply" function that does not affect base values but current values instead. Sadly my skills currently don't allow me to do this myself...