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

inject: fix Atlantean Stronghold room portals #948

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Aug 28, 2023

Resolves #227.

Checklist

  • I have read the coding conventions
  • I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

This amends the portals between rooms 74 and 12 per @SuiKaze's comments. The crash is easy to reproduce with the fly cheat if you get into this area with the flip map on. It's also entirely possibly to skip over the flipoff trigger fix highlighted below if you make a mistake and fall down, and then use a setup used by speedrunners to drop into the hole.

image

Demo of the crash and the fix.
https://www.youtube.com/watch?v=ye_jnebNu3U

The modified injection function here is also used in Midas to amend the height of the portal between rooms 9 and 13 (which is linked to texture fixes in this area). The change was needed as there are 3 different portals between these Stronghold rooms, but the previous approach only allowed targeting one per room pair.

Adds to the Atlantean Stronghold FD injection fixes by reconfiguring
the portals between rooms 74 and 12 such that the vertices are within
the room bounds in all cases. This amends the injection function for
this to allow targeting specific doors as there are 3 in this case.
Its other use case is in Palace Midas where there is only 1 portal.

Resolves LostArtefacts#227.
@lahm86 lahm86 added OG bug A bug in original game Data Requires injecting content / files that are not a part of the original game labels Aug 28, 2023
@lahm86 lahm86 added this to the 2.16 milestone Aug 28, 2023
@lahm86 lahm86 self-assigned this Aug 28, 2023
@lahm86 lahm86 requested review from rr- and walkawayy August 28, 2023 10:27
Copy link
Collaborator

@walkawayy walkawayy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!! This bug can finally be fixed and closed.

@lahm86 lahm86 merged commit a914c53 into LostArtefacts:develop Aug 28, 2023
1 check passed
@lahm86 lahm86 deleted the issue-227-stronghold-portals branch August 28, 2023 13:12
@rr- rr- added the TR1 label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Data Requires injecting content / files that are not a part of the original game OG bug A bug in original game TR1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Original TRUB levels crash on final Atlantean Stronghold jump
3 participants