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: add user controllable FOV #2178

Merged
merged 2 commits into from
Dec 31, 2024
Merged

TR2: add user controllable FOV #2178

merged 2 commits into from
Dec 31, 2024

Conversation

rr-
Copy link
Collaborator

@rr- rr- commented Dec 31, 2024

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
  • I have added a readme entry about my new feature or OG bug fix, or it is a different change

Description

I chose to always apply the vertical FOV instead of making it a setting like in TR1. Instead, I included TR2Main's PCX FOV feature. This is primarily important for the inventory ring and cutscenes, which have fixed FOV values that players can't adjust themselves. I've been careful not to break the cutscenes, but it would be helpful to have them reviewed.

@rr- rr- added OG bug A bug in original game Feature New functionality Enhancement Improvement of an existing feature TR2 labels Dec 31, 2024
@rr- rr- self-assigned this Dec 31, 2024
@rr- rr- requested review from a team as code owners December 31, 2024 00:16
@rr- rr- requested review from lahm86, walkawayy and aredfan and removed request for a team December 31, 2024 00:16
Copy link

github-actions bot commented Dec 31, 2024

@Richard-L
Copy link
Collaborator

Will try to do more testing after some sleep, but:

  • What's TR2Main's PCX feature?
  • Is there, or do you plan to enable, a way from the config tool, or in-game sunglasses settings, to set the PS1 FOV? This new FOV in the build is a definite improvement, but the ring menu and Lara with weapon equipped feel rather tiny to my PS1 eyes.

@rr-
Copy link
Collaborator Author

rr- commented Dec 31, 2024

Will try to do more testing after some sleep, but:

  • What's TR2Main's PCX feature?

It slightly enlarges the game by using a different height option. I recommend enabling this.

  • Is there, or do you plan to enable, a way from the config tool, or in-game sunglasses settings, to set the PS1 FOV?

I read this as two questions, re: better UI – yes this is #1615, re: PS1 FOV – why not? But please do try the PCX FOV and LMK if it's different, ideally with screenshots of the title screen.

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.

Looks great! Thank you. 👍

@aredfan
Copy link
Collaborator

aredfan commented Dec 31, 2024

I did end up finding an issue after all. It's possible for the /set pcx command to stop working after Lara uses a key on a keyhole. It can also happen with a puzzle hole.

  1. /play 12
  2. /tp keyhole
  3. use the all items cheat
  4. use the key
  5. /set pcx 0 or /set pcx 1

@Richard-L
Copy link
Collaborator

How do you enable PCX?

Did I miss it in the config tool?

@rr-
Copy link
Collaborator Author

rr- commented Dec 31, 2024

How do you enable PCX?

Did I miss it in the config tool?

For now, no UI for this – /set or the config file. I'll add that to the changelog.
It won't make its way to the config tool – it's supposed to be easily changeable from the game, and we'll get to that in #1615 but I cannot work on everything at once.

Copy link
Collaborator

@lahm86 lahm86 left a comment

Choose a reason for hiding this comment

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

Looks great. My only thought would be to make the pcx setting enabled by default.

@rr- rr- force-pushed the issue-2002-user-fov branch 2 times, most recently from 91909fc to 562f4b5 Compare December 31, 2024 09:26
@rr- rr- force-pushed the issue-2002-user-fov branch from 562f4b5 to 80c391a Compare December 31, 2024 09:29
@rr-
Copy link
Collaborator Author

rr- commented Dec 31, 2024

@aredfan @lahm86 fixed.
Also, found an issue where 3D pickups would pick up user FOV, which for extreme values would look really off. Fixed that as well.

I changed the order of the inventory ring initialization so it'd be good to test its interaction with game music, the spinout sound, and other side effects.

@rr- rr- requested review from aredfan and lahm86 December 31, 2024 09:29
@rr- rr- merged commit 3e2d54c into develop Dec 31, 2024
7 checks passed
@rr- rr- deleted the issue-2002-user-fov branch December 31, 2024 12:59
@Richard-L
Copy link
Collaborator

For now, no UI for this – /set or the config file. I'll add that to the changelog. It won't make its way to the config tool – it's supposed to be easily changeable from the game, and we'll get to that in #1615 but I cannot work on everything at once.

There's still no table anywhere with all possible console commands is there? I tried a couple and after some failures worked out how to turn PCX on/off. On seems to look like OG?

I also tried to set the FOV to 60 like you can in TR1X' config file, but I didn't manage to guess the right console command here.

Tbh the console is a little frustrating due to the intended guesswork, and due to it not being possible to type / on my keyboard. But it seems to work without it still.

@walkawayy
Copy link
Collaborator

Here's all the commands. Looks like the FOV one isn't in there yet though.

https://github.com/LostArtefacts/TRX/blob/develop/docs/tr2/COMMANDS.md

@Richard-L
Copy link
Collaborator

Thanks a lot, that's helpful.

@rr-
Copy link
Collaborator Author

rr- commented Jan 1, 2025

Here's all the commands. Looks like the FOV one isn't in there yet though.

https://github.com/LostArtefacts/TRX/blob/develop/docs/tr2/COMMANDS.md

The command for it is called /set, I'll add it to the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvement of an existing feature Feature New functionality OG bug A bug in original game TR2
Projects
Archived in project
5 participants