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

Support for 120hz position update #97

Open
lomeat opened this issue Aug 15, 2023 · 13 comments · May be fixed by #98
Open

Support for 120hz position update #97

lomeat opened this issue Aug 15, 2023 · 13 comments · May be fixed by #98

Comments

@lomeat
Copy link

lomeat commented Aug 15, 2023

I have a macbook m1 pro with 120hz and display with 144hz screen update, so the current position updating looks like "lagging" because of 60 "fps".

The best option is take the monitor refreshing latency and use it to delay calculations (as a delta)
Or you can add variants to menu for choosing user frequency

@RealA10N
Copy link

RealA10N commented Sep 28, 2023

Hi! I have the same issue.
It seems like modifiying the following two lines in the source code to something like 1/144 will do the trick.

static const double kMoveFilterInterval = 0.02;
static const double kResizeFilterInterval = 0.04;

IMO a reasonable deafult value will be 0.01 which corresponds to 100hz.

@RealA10N
Copy link

Here is the version I compiled, it works great, much smooooooother!
Easy.Move+Resize.app.zip
(unzip, then drap to applications, then launch).

@RealA10N RealA10N linked a pull request Sep 28, 2023 that will close this issue
@mularkey
Copy link

mularkey commented Oct 5, 2023

This! Incredible, thank you!

@Arecsu
Copy link

Arecsu commented Oct 22, 2023

Indeed. It works so much better @RealA10N, thank you! 😊

The stutter in the main release also happens in a 60hz screen. It looks like 20fps, maybe less. I'm using a Macbook Air with Sonoma

@jjolmo
Copy link

jjolmo commented Oct 24, 2023

@RealA10N Works great!. It could be ideal to be able to configure the refresh rate via options

@seanlzq-hm
Copy link

@RealA10N Works great!. It could be ideal to be able to configure the refresh rate via options

Yes this would be great. I'm using a 144hz monitor as well and I'm notifcing the jankiness still

@RealA10N
Copy link

RealA10N commented Dec 5, 2023

@RealA10N Works great!. It could be ideal to be able to configure the refresh rate via options

Yes this would be great. I'm using a 144hz monitor as well and I'm notifcing the jankiness still

Hi, you can change the constant to 1/144 or even 1/200 and compile the project yourself 🙂

I'm not the maintainer of this project, it's just a quick fix

@seanlzq-hm
Copy link

@RealA10N Works great!. It could be ideal to be able to configure the refresh rate via options

Yes this would be great. I'm using a 144hz monitor as well and I'm notifcing the jankiness still

Hi, you can change the constant to 1/144 or even 1/200 and compile the project yourself 🙂

I'm not the maintainer of this project, it's just a quick fix

I would if I knew how, I'm just the average consumer with no app development knowledge 😅

@csumpasd
Copy link

csumpasd commented Dec 5, 2023

@seanlzq-hm you have to download the project (for example, through the releases page, it's a zip file), then open the contained .xcodeproj file in xcode, find the EMRAppDelegate.h file on the right and edit the variables inside, and then do the steps at the bottom of the readme file, starting with "In Xcode...". You should have the app in your homedir in a folder.

@seanlzq-hm
Copy link

seanlzq-hm commented Dec 9, 2023

Edited and recompiled for my 144hz monitor...

Easy Move+Resize.zip

So far so good. Thanks for the help @csumpasd and @RealA10N !

@bcalik
Copy link

bcalik commented Dec 23, 2023

Edited and recompiled for my 144hz monitor...

Easy Move+Resize.zip

So far so good. Thanks for the help @csumpasd and @RealA10N !

This works awesome on my M1 Macbook Pro (120hz) with any 144hz or 60hz displays together! Its super smooth.

@cristianrgreco
Copy link

@dmarcotte it'd be great to make this the default or make it user configurable and get a release, i'm a bit nervous about downloading and running apps attached to issue comments

@cschep
Copy link

cschep commented Apr 12, 2024

I've opened #102 to get moving on a menu item for this. Would love any thoughts!

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

Successfully merging a pull request may close this issue.

10 participants