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

movement optimisations #82

Merged
merged 13 commits into from
Jan 30, 2025
Merged

movement optimisations #82

merged 13 commits into from
Jan 30, 2025

Conversation

myrrc
Copy link
Collaborator

@myrrc myrrc commented Jan 26, 2025

  • Don't use util functions as they're slow, retain them only for "big" items motions
  • Debug mode for tester script, less output for single test, less sleep between commands
  • New movements test
  • curl instead of wget in tester
  • sws bump to 2.14.0.3
  • new FirstItemStart item mapped to ^
  • no windows backslashes \ in rk.lua (lua guarantees to match forward slashes on every platform)
  • Don't show .RPP files in Github diffs
  • custom_actions folder removed in favour of movements.lua
    Resolves input latency #81

@0atman
Copy link

0atman commented Jan 27, 2025

Thank you so much for this - Sadly this did not fix my problem, movement is still very slow.

I've poked around and found something that seems weird, but might be a clue:

hitting <SPC> to open the menu is INSTANT. hitting <ESC> to close the menu and re-focus the editor has THE SAME LAG as item movement.
That's strange, right?

@0atman
Copy link

0atman commented Jan 27, 2025

I have just done what I should have done at the start, reinstalled reaper fresh, then installed reaper-keys. No lag. I'll compare my old config to this blank config, see what's gone wrong!

@myrrc myrrc changed the title faster b,w,e bindings movements optimisations Jan 29, 2025
@myrrc myrrc changed the title movements optimisations movement optimisations Jan 30, 2025
@myrrc myrrc merged commit a646bf9 into master Jan 30, 2025
1 check passed
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 this pull request may close these issues.

input latency
2 participants