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

misaligned stairs in subway station #68957

Closed
Vaskritaya opened this issue Oct 28, 2023 · 4 comments · May be fixed by #68496
Closed

misaligned stairs in subway station #68957

Vaskritaya opened this issue Oct 28, 2023 · 4 comments · May be fixed by #68496
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility stale Closed for lack of activity, but still valid.

Comments

@Vaskritaya
Copy link
Contributor

Vaskritaya commented Oct 28, 2023

Describe the bug

The stairs in the aboveground section of this subway station are misaligned with the section under it, leading to some strange behaviors. I was able to attack enemies in the maintenance room from the stairway above, since I could "see" down to them, despite them not actually being present if I peeked or stepped down.

Attach save file

Swain-trimmed.tar.gz

Steps to reproduce

using attached save:

  1. load in. try going up/down the subway stairs. issue will become obvious.
  2. note the maintenance room on the first sublevel, with bile splatter, brute corpse, etc. these enemies were killed from the northernmost three stairways of the aboveground station.

from fresh save, untested:

  1. find station facing a different direction than the subway lines in worldgen?

Expected behavior

Stairs should obviously be aligned. Perhaps variants of aboveground stations should exist for the purposes of having stairs match to the underground sections?

Screenshots

image
image

Versions and configuration

  • OS: Linux
    • OS Version: LSB Version: n/a; Distributor ID: Arch; Description: Arch Linux; Release: rolling; Codename: n/a;
  • Game Version: ce449ad [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth],
    Bionic Slots [cbm_slots]
    ]

Additional context

The station was actually generated in an earlier experimental, about a week or so ago, but unless I missed something, this wasnt noticed or fixed between then and now.

Also, on further inspection, it seems like the subway spawned somewhere where it COULD HAVE faced the proper direction. If it faced the road to its north, everything would have lined up. Perhaps this is relevant?

@Vaskritaya Vaskritaya added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Oct 28, 2023
@ZhilkinSerg
Copy link
Contributor

Related to #49162 and #45180

@AmarinReyny
Copy link

AmarinReyny commented Oct 29, 2023

I've noticed in my own playthroughs that the sublevel for subway stations is always generated facing North, regardless of what directions the other Z levels are facing.

Copy link
Contributor

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.

@github-actions github-actions bot added the stale Closed for lack of activity, but still valid. label Nov 28, 2023
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 28, 2023
@Procyonae
Copy link
Contributor

#68496 should fix this when I get back to it

@Procyonae Procyonae mentioned this issue Dec 29, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S1 - Need confirmation) Report waiting on confirmation of reproducibility stale Closed for lack of activity, but still valid.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants