#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 and the library aura (according to TBD tooltips and to design docs).
- Moreover if we change multipliers straightly in the template, it is well taken into account in the structure tree.
Not done :
- I think Petra stuff can 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).