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

Fixed bandits block autonoting #36397

Merged
merged 3 commits into from
Dec 24, 2019
Merged

Fixed bandits block autonoting #36397

merged 3 commits into from
Dec 24, 2019

Conversation

Night-Pryanik
Copy link
Contributor

@Night-Pryanik Night-Pryanik commented Dec 24, 2019

Summary

SUMMARY: Content "Fixed bandits block autonoting due to a typo."

Purpose of change

Overmap quickly gets cluttered with autonotes of "atmosphere" or "ambient" map extras (such as groves, dead vegetation, roadworks, or casings) as player travels around the world. I think these map extras are not worth autonoting.
Fixed bandits block autonoting due to a typo.

Describe the solution

Remove autonote generation from mx_crater, mx_fumarole, mx_roadworks, mx_mayhem, mx_grove, mx_shrubbery, mx_clearcut, mx_pond, mx_clay_deposit, mx_dead_vegetation, mx_point_dead_vegetation, mx_burned_ground, mx_point_burned_ground, and mx_casings map extras.
Also fixed bug in autonoting the bandits block map extra.

Describe alternatives you've considered

None.

Testing

Started a new game, checked if there are any errors (none observed).

Additional context

None.

@Night-Pryanik Night-Pryanik added Info / User Interface Game - player communication, menus, etc. Map / Mapgen Overmap, Mapgen, Map extras, Map display labels Dec 24, 2019
@snipercup
Copy link
Contributor

I don't agree, they are useful to me, the player should choose a 'level' of detail on autonotes and put the ambient in a 'high detail' level

@sztosz
Copy link
Contributor

sztosz commented Dec 24, 2019

You can always turn them off after encountering them for the first time.

I don't like to have them noted, so I turn them off, but they may be useful for others, and since they are so easy to turn off, I'd vote to leave it as it is without proposed changes.

@ZhilkinSerg
Copy link
Contributor

We can change autonote from Boolean to some kind of map extra category enum and store in global/world options which of these categories should have auto-notes enabled.

@Night-Pryanik Night-Pryanik changed the title Remove autonotes from "ambient" map extras Fixed bandits block autonoting Dec 24, 2019
@Night-Pryanik
Copy link
Contributor Author

Ok, I reverted most of the changes. I totally forgot about Autonotes Manager. It's only bugfix now.

@Night-Pryanik Night-Pryanik added the <Bugfix> This is a fix for a bug (or closes open issue) label Dec 24, 2019
@Night-Pryanik
Copy link
Contributor Author

Though I want to mention that we don't autonote some of map extras, for example patches of long grass. I think there should be a consistency for "ambient" map extras.

@ZhilkinSerg
Copy link
Contributor

Though I want to mention that we don't autonote some of map extras, for example patches of long grass. I think there should be a consistency for "ambient" map extras.

I agree - we need to review autonotes for map extras before 0.E. Player who would like to enable notes for more extras can just use copy-from in a mod.

@ZhilkinSerg ZhilkinSerg merged commit d588059 into CleverRaven:master Dec 24, 2019
@Night-Pryanik Night-Pryanik deleted the remove-autonotes-from-ambient-map-extras branch December 24, 2019 10:06
@kevingranade
Copy link
Member

I do think what we really want here is some kind of filtering mechanism so that the player can get at the things they are interested in.

The category/level of detail idea seems promising.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) Info / User Interface Game - player communication, menus, etc. Map / Mapgen Overmap, Mapgen, Map extras, Map display
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants