Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LTM] Reverse redefinition of farmland terrain as plains #662

Open
zijistark opened this issue Dec 15, 2019 · 0 comments
Open

[LTM] Reverse redefinition of farmland terrain as plains #662

zijistark opened this issue Dec 15, 2019 · 0 comments

Comments

@zijistark
Copy link
Member

I'm assigning this to @DelnarErsike and myself, because @Leybrook (kotten) is not yet assignable due to not being on EMFTeam.

I quote from Slack:

@Leybrook

Due to the expansion of farmland, I needed it to be visual rather than affect the game mechanics too much, so I redefined farmland to work as plains
the only difference between plains and farmland is "max_attrition = -0.01" being added on the latter
and the provinces would get less attrition as they develop naturally anyhow
so this was the tradeoff for visual expansion (edited)
italian provinces get farmland due to history files

@DelnarErsike

Re: the differences, traits can add affinity for one but not the other, and events and effects can trigger off of one terrain type but not the other (e.g. in the current EMF beta, farmlands give less > Nomad population per empty holding than plains because the suitability between sedentary and nomadic lifestyles is much more in sedentary's favor in farmlands than they are in plains).
But my main concern here is actually the interaction between visuals and mechanics. CK2 doesn't have a simple terrain map mode, so seeing farmland-like terrain graphics can more easily lead people to > believe that the province is a farmland one (at a glance).
Which is true for cases like provinces in Italy and Denmark (because SWMH has manually set province terrain types to farmland there), but not for ones in East Anglia or Normandy.

@Leybrook

Ill change it then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants