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

Add option to ignore local authorities opinion #6416

Closed
DorpsGek opened this issue Jan 27, 2016 · 9 comments
Closed

Add option to ignore local authorities opinion #6416

DorpsGek opened this issue Jan 27, 2016 · 9 comments
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

Milek7 opened the ticket and wrote:

Patch for trunk in attachment

Attachments

Reported version: trunk
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6416
@DorpsGek
Copy link
Member Author

Milek7 wrote:

fixed indentation and added some comments

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14116

@DorpsGek
Copy link
Member Author

frosch wrote:

Why are you adding a new setting instead of using the existing "town council attitude" one?
(Current values: permissive, tolerant, hostile; new value: submissive/cheating)

What is the meaning of the existing one, when disabling via the new setting?


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14117

@DorpsGek
Copy link
Member Author

Milek7 wrote:

It was just easier to add this, and I do not want to mess with other code. With this setting the rating calculation stays working, but it is not checked when destroying/building station. When adding an extra option to "town council attitude" it should completlely remove rating calculation, visiblity in "Local authorities" window and bribing function.

If it is important to not add new setting, i can submit new patch.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14118

@DorpsGek
Copy link
Member Author

Alberth wrote:

It's not just important, it's a hard requirement. New code must be fully integrated into the code base.

You don't want to have multiple settings to achieve one thing (eventually leading to set this option to "yes", that one to "ignore other one", and the the third one to "yes").


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14125

@DorpsGek
Copy link
Member Author

Milek7 wrote:

No, this patch don't make multiple settings to achieve one thing. It disables blocking capabilities of local authority, but no whole rating calculation.
Copy-paste from irc:
while disabling rating calculation, what value use in GSTown.GetRating?
ignoring don't breaks anything, disabling can broke something relying on that
and it even shouldn't be disabled, because gamescript using town rating to calculate growth is very good
but this patch is for that local authority can't block building/destroying
not disabling rating completly


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14127

@DorpsGek
Copy link
Member Author

dp wrote:

I think main source of confusion here is that town_council_tolerance affects multiple things:

  1. Max town noise level
  2. Speed of noise level reduction with distance
  3. Min rating required for demolishing stuff in town
    Also, notably, it does NOT affect
  4. Town rating calculation (and thus cb scripts)
  5. Min rating for placing stations (that's always RATING_VERYPOOR)

And main issue with authority rating is it's blocking behavior (3 and 5).
So I'd suggest splitting airport noise in separate setting and making tolerance affect station placement.
That way adding ignorant/cheating attitude will allow to disable blocking aspect without breaking anything else.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14128

@DorpsGek
Copy link
Member Author

dp wrote:

Oh, and also ignoring 3 should be removed from magic bulldozer effects imo if this is implemented.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14129

@DorpsGek
Copy link
Member Author

DorpsGek commented Sep 3, 2017

andythenorth wrote:

This is IMHO a boring thing to add to the core game.

The local authority mechanic is a core mechanic of the original gameplay.

Completely nerfing it is really dull.

Does that matter?

Yes.

Patches need reviewed and tested before they can be committed.

That's work.

Working on dull things is...dull.

So dull things have very low chance of getting done.

Even if there were 50 people with commit rights, this wouldn't change.

Nobody wants to be the one adding dull stuff.

Nobody wants 49 other people asking 'wtf did you do that for?'

So eh.

Town ratings have long been a problem.

Is there a better way? :)


This comment was imported from FlySpray: https://bugs.openttd.org/task/6416#comment14743

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay labels Apr 7, 2018
@TrueBrain
Copy link
Member

This sounds more like a cheat than anything else. I do not see a need for this in the core game currently. Sorry, but thank you for the patch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

2 participants