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

Tacoma Commune faction missions are still hard-coded and require an unrotated map #33525

Closed
TJKing293 opened this issue Aug 24, 2019 · 6 comments · Fixed by #57548
Closed

Tacoma Commune faction missions are still hard-coded and require an unrotated map #33525

TJKing293 opened this issue Aug 24, 2019 · 6 comments · Fixed by #57548
Assignees
Labels
<Bug> This needs to be fixed Missions Quests and missions NPC / Factions NPCs, AI, Speech, Factions, Ownership (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@TJKing293
Copy link
Contributor

Describe the bug

When turning in the "collect 300 salt" quest from the ranch foreman everything is normal until I exit the dialog menu. Then the foreman and the other npc with him vanish along whith the tractor. I get this

DEBUG : Tried to access invalid map position at grid (172,180,0)

FUNCTION : size_t map::get_nonant(const tripoint&) const
FILE : src/map.cpp
LINE : 8075

Steps To Reproduce

attempt to complete the mission

Expected behavior

To not go away
save_file.zip

Versions and configuration

  • OS: Windows 10
  • Game Version: 0.D-6936-ge44b42
  • Graphics version:Tiles
  • Mods loaded:
    . "dda",
    "no_npc_food",
    "No_Rail_Stations",
    "Mining_Mod",
    "hydroponics",
    "fast_healing",
    "alt_map_key",
    "More_Survival_Tools",
    "no_mutagen",
    "No_Acid_Zombies",
    "No_Big_Zombies",
    "No_Explosive_Zombies",
    "No_Fungi",
    "no_filthy_clothing",
    "no_reviving_zombies",
    "novitamins",
    "StatsThroughSkills"

Additional context

Add any other context about the problem here.

@alkemann alkemann added (S1 - Need confirmation) Report waiting on confirmation of reproducibility <Bug> This needs to be fixed Missions Quests and missions NPC / Factions NPCs, AI, Speech, Factions, Ownership labels Aug 25, 2019
@smackdabinthemiddle
Copy link

I came across the exact same problem, in build 9547, Windows x64.

The Foreman and Carpenter, along with the Plow Tractor vanish from the barn for a couple of turns. The Tractor returns (after 5 min of waiting, with the diesel fuel dropped on the floor, while the NPCs are teleported elsewhere, next to a Carpenter NPC, which in my save is a couple of tiles away from the farm itself.
Here's the save if it's of any use.

@SunshineDistillery
Copy link
Contributor

Are your ranch camps rotated? As in, is the ranch house located in a spot other than the south side?
It's probably similar to #32424 except with NPCs

@smackdabinthemiddle
Copy link

In my case, yes. The ranch is north of the field, and the garage east of the ranch.

@TJKing293
Copy link
Contributor Author

It's oriented house north pond south. So yeah it is.

@kevingranade kevingranade added this to the 0.E milestone Sep 6, 2019
@mlangsdorf mlangsdorf self-assigned this Sep 6, 2019
@ZhilkinSerg ZhilkinSerg added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Sep 18, 2019
@mlangsdorf
Copy link
Contributor

The Tacoma Commune upgrade missions that are still hardcoded will update the map in strange ways if the ranch is rotated.

I think the short term solution is to make the refugee ranch map non-rotatable.

mlangsdorf added a commit to mlangsdorf/Cataclysm-DDA that referenced this issue Sep 19, 2019
Addresses CleverRaven#33525

A fair bit of the Tacoma Commune missions still use hard coded C++
functions to update the map, and those functions do not understand
map rotation at all.  Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new
construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go
through and move the remaining Tacoma Commune missions into JSON.
ZhilkinSerg pushed a commit that referenced this issue Sep 19, 2019
Addresses #33525

A fair bit of the Tacoma Commune missions still use hard coded C++
functions to update the map, and those functions do not understand
map rotation at all.  Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new
construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go
through and move the remaining Tacoma Commune missions into JSON.
@mlangsdorf mlangsdorf removed this from the 0.E milestone Sep 19, 2019
@mlangsdorf
Copy link
Contributor

Removing this from the 0.E tracker - we have a work-around that's good enough for 0.E but will need to go back and fix it properly later.

misterprimus pushed a commit to misterprimus/Cataclysm-DDA that referenced this issue Sep 21, 2019
Addresses CleverRaven#33525

A fair bit of the Tacoma Commune missions still use hard coded C++
functions to update the map, and those functions do not understand
map rotation at all.  Completing those missions will cause all
kinds of weird errors as NPCs teleport around the map and new
construction is built into old construction.

Make the ranch camp maps unable to rotate until I have time to go
through and move the remaining Tacoma Commune missions into JSON.
@mlangsdorf mlangsdorf changed the title vanishing npc's when mission is completed Tacoma Commune faction missions are still hard-coded and require an unrotated map Sep 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Missions Quests and missions NPC / Factions NPCs, AI, Speech, Factions, Ownership (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants