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

tr2/inject: add item and room editing support #1970

Merged
merged 5 commits into from
Nov 28, 2024

Conversation

lahm86
Copy link
Collaborator

@lahm86 lahm86 commented Nov 28, 2024

Resolves #1716.
Resolves #1939.
Resolves #1963.

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 fixes a few more FD issues as detailed in the issues. I've bundled the door fixes for Ice Palace and Floating Islands in as FD fixes too, even though they involve repositioning items. Given the issue they cause, it feels more natural coming under the FD fix config setting.

The item editing function will also be used later for #1613.

Ref: LostArtefacts/TRXInjectionTool#13

This adds injection support for changing item positions and angles.
This fixes doors in Ice Palace and Floating Islands that are
positioned/rotated incorrectly such that they cause an invisible wall
on the wrong side.

Resolves LostArtefacts#1963.
This extends the Gong Hammer's pickup trigger into the adjacent tile to
ensure it is detected regardless of Lara's position.

Resolves LostArtefacts#1716.
This adds a new floor data fix type to allow editing room properties,
currently limited to room flags.
This floods room 98 in Wreck of the Maria Doria.

Resolves LostArtefacts#1939.
@lahm86 lahm86 added OG bug A bug in original game Data Requires injecting content / files that are not a part of the original game TR2 labels Nov 28, 2024
@lahm86 lahm86 self-assigned this Nov 28, 2024
@lahm86 lahm86 requested review from a team as code owners November 28, 2024 21:22
@lahm86 lahm86 requested review from rr-, walkawayy and aredfan and removed request for a team November 28, 2024 21:22
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 beautifully. Thank you very much for tackling these 3 issues. 👍

@lahm86 lahm86 merged commit 67a7531 into LostArtefacts:develop Nov 28, 2024
6 checks passed
@lahm86 lahm86 deleted the issue-1967-tr2-injection branch November 28, 2024 22:05
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 TR2
Projects
Archived in project
3 participants