-
Notifications
You must be signed in to change notification settings - Fork 41
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
Hovered cursor is slow to scroll with non-latin characters #131
Comments
I cannot reproduce. Can you share some more information like operating system and mpv version? After removing the files with non-latin characters it works faster? |
Debian Sid and mpv 0.36.0. |
If you make Or can you try if it gets faster if you change function parse_filename(string, name, index)
local base = tostring(plen):len()
return string:gsub("%%pos", string.format("%0"..base.."d", index+1))
:gsub("%%name", name)
end I'm wondering if the gsubs are causing some performance issues with some characters. Also can you try using this title:
I wonder if it's the Also could you try using another font if that solves the issue? Then it could be some issue in font rendering rather than in this script. |
Same issue.
The default font I'm using for OSD is |
I think 0xFF09 is the Maybe this discussion has some solutions https://www.reddit.com/r/archlinux/comments/8ciooy/support_for_fullwidth_latin_letters/ I'm not sure if there is anything I can do on the script level to fix this issue. It might be also useful to report to mpv repository, as they could probably do some caching and more performant fallbacks on missing characters. |
I think this cause performance issues, do you know how to stop it (except to close the playlist)?
Thanks for the info, I tested |
Here is a thread about similar topic, mentioning the slowdown as well. mpv-player/mpv#8919 I think the solution is either in the system or optimizations in mpv/libass. I recommend opening or looking for an issue in those repositories if you cannot resolve the issue. I'll close this as there is not really anything this script can do to avoid this issue, and my knowledge on the topic is limited. |
Playlist to test:
Does not happen when paused.
The text was updated successfully, but these errors were encountered: