Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#4429 closed defect (wontfix)

[PATCH] Smith aura for citizen soldier

Reported by: fatherbushido Owned by:
Priority: Should Have Milestone:
Component: UI & Simulation Keywords: patch
Cc: scythetwirler, mimo, elexis Patch:

Description

That was written in many places. Sometimes (often) players (new or old) asks if it works. I don't know if we should implement it. If so:

  • should we use an aura
  • or work directly in a component

The first solution works pretty well but I wonder if it's not too bad from the perfomance point of view.

Attachments (1)

blacksmith.diff (1.4 KB ) - added by fatherbushido 7 years ago.
numbers can be obviously tweaked

Download all attachments as: .zip

Change History (7)

by fatherbushido, 7 years ago

Attachment: blacksmith.diff added

numbers can be obviously tweaked

comment:1 by elexis, 7 years ago

Why would anyone not garrison a unit in blacksmiths if we add that bonus? The significant bonus should come with a significant disadvantage. For example garrisoning a hero would have an impact on the player and force him to chose between two valid options.

comment:2 by fatherbushido, 7 years ago

(btw i used bad namming in the draft attached patch)

comment:3 by Stan, 7 years ago

Aren't units resources ? So while your unit is garrisoned it's not working elsewhere. You could do that only to champions which have a higher cost though

comment:4 by mimo, 7 years ago

I have mixed feelings about it. It looks to me quite alien to the game mecanism and i'm afraid it will generate useless micro (just before researching a tech, you"ll have to find a unit and garrison it, and then ungarrison it when finished) so that you'll end up with always keeping the unit garrisoned, and thus it will become more or less equivalent to have a new tech which decreases the tech time and which is more in the spirit of the rest of the game. In addition, why single out the blacksmith? and not also have such a thing for other structures which can research techs. So, if going in that direction, i would prefer:

  • either something more connected to research and more generic: for exemple, if you garrison units in the library (and maybe in temples for those civs which don't have library), you decrease the research time of all techs. And 10% for a single unit looks too much for me, i would favor something like 2% per garrisoned unit (with maybe only 1% in temples) limited to 10%.
  • or if we really want to single out blacksmith, link the time reduction to the number of blacksmiths rather than its garrisoned units: a second blacksmith would give a reduction of 10% of the blacksmith research time and a third one would give an additional 5% reduction. That may also become an incentive to capture blacksmiths.

comment:5 by fatherbushido, 7 years ago

Milestone: Work In ProgressBacklog
Resolution: wontfix
Status: newclosed

You confirm my first thoughts. Thanks for clarification! (and suggestions)

I will close the ticket.

comment:6 by elexis, 7 years ago

Keywords: rfc removed
Milestone: Backlog

Would add that there might be people on the forums who might give more reasoning than the trac people.

Note: See TracTickets for help on using tickets.