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

IGNORE_SOUND, NO_BASH and RETURN_TO_START_POS traits to more NPCs. #62470

Open
I-am-Erk opened this issue Nov 29, 2022 · 2 comments
Open

IGNORE_SOUND, NO_BASH and RETURN_TO_START_POS traits to more NPCs. #62470

I-am-Erk opened this issue Nov 29, 2022 · 2 comments
Labels
<Bugfix> This is a fix for a bug (or closes open issue) <Enhancement / Feature> New features, or enhancements on existing NPC / Factions NPCs, AI, Speech, Factions, Ownership <Suggestion / Discussion> Talk it out before implementing

Comments

@I-am-Erk
Copy link
Contributor

Is your feature request related to a problem? Please describe.

#62439 added the ability to give static NPCs mutations that stop them from investigating sounds or bashing terrain, and tell them to return to their locations when they are done other stuff.

Solution you would like.

These traits should be added to most static NPCs. For now, the ones that can be recruited don't need them, although a better way to do that would be to give them these traits and then remove them when you get to the dialogue that allows you to recruit them.

Guard NPCs should probably still investigate sounds and bash terrain.

Describe alternatives you have considered.

Improving the AI to do this on its own - most of these traits should be reworked into conditional AI features instead, after stable. However, "no investigate" should remain a common one for a lot of these NPCs.

In stable, we should probably re-evaluate NPC decisions to bash a lot more, and have them start moving furniture when they would have bashed it.

Additional context

Not every NPC needs to be covered for stable but most should be.

@I-am-Erk I-am-Erk added <Enhancement / Feature> New features, or enhancements on existing NPC / Factions NPCs, AI, Speech, Factions, Ownership <Suggestion / Discussion> Talk it out before implementing <Bugfix> This is a fix for a bug (or closes open issue) labels Nov 29, 2022
@I-am-Erk I-am-Erk added this to To-do in 0.G Stable Tracker via automation Nov 29, 2022
@Maleclypse
Copy link
Member

Do we think #62493 caught the remainder that needed this?

@I-am-Erk
Copy link
Contributor Author

I-am-Erk commented Dec 6, 2022

Perhaps, but #62472 still needs to be finished off.

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) <Enhancement / Feature> New features, or enhancements on existing NPC / Factions NPCs, AI, Speech, Factions, Ownership <Suggestion / Discussion> Talk it out before implementing
Projects
None yet
Development

No branches or pull requests

2 participants