Page MenuHomeWildfire Games

[gameplay] allow upgrading cart_house to cart_apartment
Needs ReviewPublic

Authored by borg- on Tue, Jul 28, 4:37 AM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

This patch has proposal to introduce cart_apartment by tech.
Since it was implemented in a borg expansion pack, it has been asked a lot to be implemented on vanilla.
https://wildfiregames.com/forum/index.php?/topic/28557-unused-atlas-entities/&tab=comments#comment-401602

Apartment gives:

  • +50% health
  • +3 garrison capacity
  • +5 population bonus

It's basically like a small house + big house.

Positive point:
The player benefits economically by not depending of unity to build.
Better garrison for women.

Negative point:
Must wait 40 seconds, cant speed up.
Decreases the chances of wall by house.

Test Plan

Check mistakes and omissions.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

borg- created this revision.Tue, Jul 28, 4:37 AM
borg- retitled this revision from [gameplay] ADD cart_apartment to [gameplay] ADD cart_apartment by tech.

While I like the idea of enabling the cart apartment blocks (I've done so too in my 0abc mod), I'm not entirely convinced doing it via an upgrade is the best possibility from a gameplay perspective. Upgrading does not involve any workers, therefore the time is basically free. Have you considered simply adding them to the cart build list instead?
Also, shouldn't they have a city phase requirement?

cart_apartment.xml
5–8

These lines are necessary for determining repair costs and time.

Nescio retitled this revision from [gameplay] ADD cart_apartment by tech to [gameplay] allow upgrading cart_house to cart_apartment.Tue, Jul 28, 9:48 AM
borg- added a comment.Tue, Jul 28, 6:15 PM

While I like the idea of enabling the cart apartment blocks (I've done so too in my 0abc mod), I'm not entirely convinced doing it via an upgrade is the best possibility from a gameplay perspective. Upgrading does not involve any workers, therefore the time is basically free. Have you considered simply adding them to the cart build list instead?
Also, shouldn't they have a city phase requirement?

It can only be placed a build list, but occupy an icon more and we would need a new art ícone as well. If changed to the city phase, it would become practically nonexistent "bônus". One proposal is to give something different and allow a small "economic bonus" cuz not needing someone to be built. Can we increase the tech time to 35, what do you think?

borg- added a comment.Thu, Jul 30, 3:45 AM

@Nescio
What would be the appropriate values in:

<BuildTime>75</BuildTime>
  <Resources>
    <wood>225</wood>
  </Resources>
cart_apartment.xml
5–8

What would be the appropriate values?

borg- updated this revision to Diff 12977.Fri, Jul 31, 1:50 AM
borg- edited the summary of this revision. (Show Details)

Up.

  • Keep <BuildTime>80</BuildTime> <Resources> <wood>225</wood> </Resources>
  • Research time 30 --> 40.
borg- marked an inline comment as done and an inline comment as not done.Fri, Jul 31, 1:51 AM

I prefer the idea of an upgrade than using a new slot for the same function than the house.

But when the upgrade is available to the player? Does it need to perform a tech research before to access the upgrade?

What happens if another civ captures a cart house? Can it upgrade it to a cart apartment? Is that desirable?

borg- added a comment.Wed, Aug 5, 3:44 AM

What happens if another civ captures a cart house? Can it upgrade it to a cart apartment? Is that desirable?

No. Is there a way to limit the technology to cartage only?

Nescio added a comment.Wed, Aug 5, 4:20 PM

No.

There were three questions, so no to which part(s)?

Is there a way to limit the technology to cartage only?

You could consider introducing separate phase_city_cart.json and phase_town_cart.json technologies and set the <Upgrade/RequiredTechnology> to that. You should check there are no unintended consequences, though.

borg- added a comment.Wed, Aug 5, 5:39 PM

There were three questions, so no to which part(s)?

3*

You could consider introducing separate phase_city_cart.json and phase_town_cart.json technologies and set the <Upgrade/RequiredTechnology> to that. You should check there are no unintended consequences, though.

Technology can be created to enable update. A technology can only cost time like 120, or add some small cost and then decrease a little cost of upgrade for compensation. What do you think?