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

Set max_fps to 120 by default instead of 60 #6334

Open
linkarzu opened this issue Oct 29, 2024 · 4 comments
Open

Set max_fps to 120 by default instead of 60 #6334

linkarzu opened this issue Oct 29, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@linkarzu
Copy link

Is your feature request related to a problem? Please describe.
I recently reviewed the Ghostty terminal emulator, a couple of days ago, and my only "complaint" we could say about WezTerm is that it feels more laggy, choppy, and delay-i compared to Ghostty, I've been using WezTerm like that for a month, and also when I compared it to Kitty that was my only "complaint" or let's call it feedback.

Once done with my YouTube, which can be found here someone was kind enough to let me know in the comments that WezTerm was capped at 60FPS but setting it to 120FPS made it better, I had searched for settings I could change in the past to make this better, but had no luck, so yesterday I searched for FPS and found this max_fps parameter, which seems insignificant, so I pumped it to 120 and Voilá, that did it.

All the complaints I had about WezTerm are gone, and I feel it's now as fast as Ghostty and Kitty

Describe the solution you'd like
max_fps = 120 to be the default
If not possible, I'd like to understand the reasons behind the decision, as I've read multiple other comments that they experience the same "issue" with WezTerm, it's "slower" than kitty, when in fact it's not

Additional context
Here's my other video in which I reviewed WezTerm like a month ago
My take yesterday on the Ghostty video was not fair, because WezTerm is as fast, so I'll release a follow up video soon

I love WezTerm, having this small feature would make it perfect for a lot of people, and would like to understand why it's not the default

@linkarzu linkarzu added the enhancement New feature or request label Oct 29, 2024
@realhackcraft
Copy link

As to why it's not the default, there are displays with only 60 fps. Setting it to 120 fps by default would consume much more power on those systems for rendering extra frames.

However, I think a good solution is that the default could auto detect the frame rate of the display.

@linkarzu
Copy link
Author

I use an external monitor, it's an asus PA278CV, and it's a 60Hz monitor, nothing fancy, do Hz relate to FPS? (sorry but I have no idea about video stuff at all)
My point is, even if my monitor is 60Hz, setting WezTerm to 120 FPS makes it run way smoother and the difference feels like night and day

@realhackcraft
Copy link

realhackcraft commented Oct 31, 2024

60HZ does in fact mean 60fps.

Hertz is a unit to measure one (cycle, event, frame, whatever) per second.

120FPS on 60HZ should be the exact same as 60FPS.

I looked up your display and it says it runs at 75HZ. If you run 60FPS on a 75HZ, you will necessarily get some inconsistent and slightly jittery frames.

Try seeing if you can see the difference between 120FPS and 75FPS. If anything, 75FPS on your monitor should be even smoother than 120FPS.

@linkarzu
Copy link
Author

Appreciate you taking the time to explain, and you're right, I thought my monitor was 60Hz, so maybe those additional 15Hz are the ones making the difference, I'm out at the moment, but will try setting it up for 75Hz when I'm back in the next days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants