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

tr1/data: fix various camera positions #2020

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Dec 13, 2024

Resolves #1317.
Resolves #2019.

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 adds workarounds for cameras in Return to Egypt and Temple of the Cat to resolve LOS issues introduced by 4.1 (the issue being the cameras are positioned where walls obstruct their view, but 4.1 removed LOS checks on fixed cameras to resolve many other problems).

The camera in Natla's Mines is also shifted slightly when moving the boat so that you can't see the already flipped state.

This is low priority but I'm aiming to tidy up some injection loose ends before we continue with the overall format refactor.

Ref: LostArtefacts/TRXInjectionTool#18

This allows injections to shift fixed cameras.
This adjusts some cameras in Return to Egypt and Temple of the Cat to
resolve LOS issues.

Resolves LostArtefacts#1317.
This shifts the camera in Natla's Mines that looks at the boat while
it's moving, so to avoid seeing the flip state.

Resolves LostArtefacts#2019.
@lahm86 lahm86 added TRX bug A bug with TRX OG bug A bug in original game Data Requires injecting content / files that are not a part of the original game TR1 labels Dec 13, 2024
@lahm86 lahm86 self-assigned this Dec 13, 2024
@lahm86 lahm86 requested review from a team as code owners December 13, 2024 11:46
@lahm86 lahm86 requested review from rr-, walkawayy and aredfan and removed request for a team December 13, 2024 11:46
Copy link

Copy link
Collaborator

@aredfan aredfan left a comment

Choose a reason for hiding this comment

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

These fixes are working perfectly, thank you. 👍

@lahm86 lahm86 merged commit 74d8a60 into LostArtefacts:develop Dec 13, 2024
6 checks passed
@lahm86 lahm86 deleted the camera-fixes branch December 13, 2024 12:41
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 TRX bug A bug with TRX
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

OG bug: flipmap is visible while boat is moving in Mines Trapdoor camera issue in Temple of the Cat
3 participants