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

TR2X bug: visible overlap of rooms in The Deck #1853

Closed
aredfan opened this issue Nov 8, 2024 · 3 comments · Fixed by #1866
Closed

TR2X bug: visible overlap of rooms in The Deck #1853

aredfan opened this issue Nov 8, 2024 · 3 comments · Fixed by #1866
Assignees
Labels
TR2 TRX bug A bug with TRX

Comments

@aredfan
Copy link
Collaborator

aredfan commented Nov 8, 2024

Regression since: 0.6


Steps to reproduce:

  1. /play 10
  2. /tp 51 4 48
  3. move the camera until you can see the rooms overlapping

20241108_195122_The_Deck

@lahm86
Copy link
Collaborator

lahm86 commented Nov 8, 2024

This is a strange one. It seems like a data issue - room 17 has a visibility portal into room 104 (which is clearly wrong, must have been a level editing bug), and in turn because that is linked to room 80, room 80 is drawn. The portal is marked by the waypoints here, just imagine it pointing downwards to room 104.

image

I'm not sure where the regression is if this doesn't happen in OG. Ideally we would fix the data and the regression itself.

@aredfan aredfan added the Data Requires injecting content / files that are not a part of the original game label Nov 9, 2024
@aredfan
Copy link
Collaborator Author

aredfan commented Nov 9, 2024

Thanks for providing a detailed breakdown there Lahm.

Btw this issue also manifests in the upper end of the level where Lara picks up the last key.

20241109_072217_The_Deck

@lahm86
Copy link
Collaborator

lahm86 commented Nov 9, 2024

This seems to have been caused by 6c36549. Restoring the various g_FltWinLeft = g_PhdWinMinX; //...etc calls in output.c that were removed in that commit fixes it, but there are side effects:

  • Restoring in Output_InsertRoom fixes the main problem, but 3D pickups aren't shown in-game, and the inventory is empty.
  • Restoring in Output_InsertRoom and Output_InsertPolygons fixes the problem and pickup/inventory display, but the 3D pickups appear centre-screen.

image

@rr- rr- moved this to Ready in TRX Dev tracker Nov 11, 2024
rr- added a commit that referenced this issue Nov 11, 2024
Regression from 6c36549, related to centering the 3d pickups in the UI.
Resolves #1853.
@rr- rr- moved this from Ready to In progress in TRX Dev tracker Nov 11, 2024
@rr- rr- closed this as completed in #1866 Nov 11, 2024
rr- added a commit that referenced this issue Nov 11, 2024
Regression from 6c36549, related to centering the 3d pickups in the UI.
Resolves #1853.
@github-project-automation github-project-automation bot moved this from In progress to Done in TRX Dev tracker Nov 11, 2024
@aredfan aredfan removed the Data Requires injecting content / files that are not a part of the original game label Nov 11, 2024
@rr- rr- self-assigned this Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR2 TRX bug A bug with TRX
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants