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

Items in trash cans should not be owned by a faction. #54745

Closed
haveric opened this issue Jan 24, 2022 · 8 comments
Closed

Items in trash cans should not be owned by a faction. #54745

haveric opened this issue Jan 24, 2022 · 8 comments
Labels
NPC / Factions NPCs, AI, Speech, Factions, Ownership stale Closed for lack of activity, but still valid. <Suggestion / Discussion> Talk it out before implementing

Comments

@haveric
Copy link
Contributor

haveric commented Jan 24, 2022

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

Walking around the refugee center, all of the items in the trash cans are still owned by the Free Merchants. Since they threw it away, I would expect the trash items to no longer be owned by them and be possible to take without stealing. This probably occurs in other faction related buildings as well.

Solution you would like.

Remove ownership of items in trash cans in faction specific buildings.

Proposed implementation (for the refugee center) would be replacing:

    "object": {
      "faction_owner": [
        { "id": "free_merchants", "x": [ 0, 23 ], "y": [ 0, 23 ] },
        { "id": "free_merchants", "x": [ 24, 47 ], "y": [ 0, 23 ] },
        { "id": "free_merchants", "x": [ 48, 71 ], "y": [ 0, 23 ] },
        { "id": "free_merchants", "x": [ 72, 95 ], "y": [ 0, 23 ] },
        { "id": "free_merchants", "x": [ 96, 119 ], "y": [ 0, 23 ] }
      ],
      "fill_ter": "t_floor",

with

    "object": {
      "faction": "free_merchants",
      "fill_ter": "t_floor",

And then instead of having jmapgen_faction run at the end, the default faction would get passed through to all of the other mapgen, allowing them to override if needed. Some of the override functionality is added here: #56110

Describe alternatives you have considered.

Leave it as is.

@haveric haveric changed the title Items in trash cans should not owned by a faction. Items in trash cans should not be owned by a faction. Jan 24, 2022
@LeahLuong
Copy link

It's not like garbage items are hard to come by elsewhere . . .

@NetSysFire NetSysFire added <Suggestion / Discussion> Talk it out before implementing NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Jan 25, 2022
@NetSysFire
Copy link
Member

That does not invalidate the issue.

In the Cataclysm I would imagine folks do more recycling or at least burn their trash for fuel though.

@haveric
Copy link
Contributor Author

haveric commented Jan 25, 2022

It's not like garbage items are hard to come by elsewhere . . .

No, but sometimes you need that one item that happens to be in someone's garbage. Or perhaps you started a run near the refugee camp and don't have those "garbage" items yet. There's always other ways to acquire items; this could just be another one.

In the Cataclysm I would imagine folks do more recycling or at least burn their trash for fuel though.

This feels like something they would have setup elsewhere for specific recycling of materials, perhaps even something that could be added down the road through quests?

@NetSysFire
Copy link
Member

I would say the refugee center in general is very underprepared, so they would not have this. I think the lore for that it is a bunch of office employees being brought there in a bus. They do not have any outstanding survivalist skills, they are ordinary civilians who maybe do not even think about reusing their trash, so it would be fine by me to make the trash unowned but there is the complexity and actual implementation.

I have no idea how ownership is managed currently and it might be difficult to exempt something.

@haveric
Copy link
Contributor Author

haveric commented Jan 25, 2022

I would say the refugee center in general is very underprepared, so they would not have this

I would somewhat agree with this. I know they supposedly have a whole vault setup underground with more functionality (lore-wise at least), but the above area is pretty limited. I'm also not sure how useful it would be to the player to assist them in constructing one vs the other planned expansion features (defenses, farming, etc.)

I have no idea how ownership is managed currently and it might be difficult to exempt something.

My assumption is that ownership is applied generally to the whole region/map, but figured this would be something worth exploring. If someone can point me in the right direction of how ownership gets applied, I'd love to poke around the code a bit.

@stale
Copy link

stale bot commented Mar 2, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@stale stale bot added the stale Closed for lack of activity, but still valid. label Mar 2, 2022
@stale stale bot removed the stale Closed for lack of activity, but still valid. label Mar 17, 2022
@stale
Copy link

stale bot commented Apr 16, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. Please do not 'bump' or comment on this issue unless you are actively working on it. Stale issues, and stale issues that are closed are still considered.

@stale stale bot added the stale Closed for lack of activity, but still valid. label Apr 16, 2022
@Night-Pryanik
Copy link
Contributor

Closing as stale, since stalebot can't do it by itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NPC / Factions NPCs, AI, Speech, Factions, Ownership stale Closed for lack of activity, but still valid. <Suggestion / Discussion> Talk it out before implementing
Projects
None yet
Development

No branches or pull requests

4 participants