#3930 closed enhancement (fixed)
[PATCH] Modifications of Technology Resources Cost and Research Time.
Reported by: | fatherbushido | Owned by: | sanderd17 |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 21 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description (last modified by )
As noticed at wiki and in discussions on irc in 2012, the modifications of technology cost and research time is still TBD.
Attachments (14)
Change History (26)
comment:1 by , 8 years ago
Description: | modified (diff) |
---|
by , 8 years ago
Attachment: | 3930.2.diff added |
---|
by , 8 years ago
Attachment: | 3930.3.diff added |
---|
comment:2 by , 8 years ago
Keywords: | review added |
---|
- patch enables modification of technology Technology/Cost/stuff (stuff can be food, wood, stone, metal, time).
- allows local modifications by Auras
- uses some bad names (that function in GuiInterface) and it is perhaps sometimes clumsy.
- doesn't care of Petra.
by , 8 years ago
Attachment: | 3930.4.diff added |
---|
comment:3 by , 8 years ago
Keywords: | review removed |
---|
by , 8 years ago
Attachment: | 3930.5.diff added |
---|
comment:4 by , 8 years ago
Keywords: | review added |
---|
After discussions with mimo and sanderd17:
- add a mandatory TechCostMultiplier in ProductionQueue schema (set to 1 by default). I checked all production queue (and even check with scenario Units demo).
- patch enables modification of technology cost in the production queue of a building with the key TechCostMultiplier/stuff (stuff can be food, wood, stone, metal, time).
- Said modifications are well taken into account in the gui.
- I joined as example two mauryans auras (according to TBD tooltips and to design docs).
- I sorted some templates where alphabetical order was messed.
I think Petra stuff can be adressed in another ticket.
by , 8 years ago
Attachment: | 3930.7.diff added |
---|
comment:5 by , 8 years ago
Done : !
- add a mandatory TechCostMultiplier in ProductionQueue schema (set to 1 by default). I checked all production queue (and even check with scenario Units demo).
- patch enables modification of technology cost in the production queue of a building with the key TechCostMultiplier/stuff (stuff can be food, wood, stone, metal, time).
- Said modifications are well taken into account in the gui.
- I joined as example two mauryans auras (according to TBD tooltips and to design docs).
Not done :
- I think Petra stuff can be adressed in another ticket.
- Moreover if we want to change multipliers straightly in the template, it is not yet taken into account in the structure tree. It can perhaps be adressed in another ticket.
by , 8 years ago
Attachment: | 3930.8.diff added |
---|
by , 8 years ago
Attachment: | 3930_testcaseforstructuretree.diff added |
---|
testcase : change the tech food cost for athenians fortress techs
comment:6 by , 8 years ago
Might I suggest these additional changes for compatibility with the structree: http://pastebin.com/deBctCbN?
clone()
to prevent multipliers displaying on other buildings using the affected techs; and making sure techCostMultiplier
s are used with wallsets.
(Edit: Clarified brackets in diff per elexis comments on irc)
comment:7 by , 8 years ago
Assuming you want tech multipliers to apply to technologies researchable directly by units (see #4001), then might I propose the additional change http://pastebin.com/rjYSNzWz to get the multipliers to be taken into account for "Trainer Units" in the structree
?
comment:9 by , 8 years ago
I've created a new ticket #4003 with the needed changes for the AI, but these can be done afterwards as long as that the current changes only decrease the cost. So having a different ticket for the AI will ease the reviews.
by , 8 years ago
Attachment: | 3930.13.diff added |
---|
remove productionqueue of mauryan_pilar and sele_colonade
comment:11 by , 8 years ago
Keywords: | review removed |
---|---|
Type: | defect → enhancement |
The wip patch implement that and allow range, garrison, garrisonedUnits aura to modify costs of techs. I join as example two mauryans auras (according to TBD tooltips and to design docs). The patch is still a draft but it "works" (i need to check translate stuff and Petra).