-
-
Notifications
You must be signed in to change notification settings - Fork 537
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
Playback speed not working anymore #1856
Comments
I'm interested in tackling this task, but I would appreciate it if additional information could be provided. :) |
hi @killthe3vil what version? @shadow-ryu Awesome!! cant say much. some of our stuff needs maintenance when youtube's DOM / CSS changes, but html5 browser standard playback features shouldnt break and youtube player's functions are well known / might not change. in the repo a complex enhancement deciding what's music (no speed up) was just added a few days ago. This will only reach stores today or tomorrow. |
im not sure what info you want, i will just show everything i can |
ok |
@shadow-ryu Appreciated for the help!~ |
@killthe3vil can u share the local setup guide as I cant find the same. |
@shadow-ryu local setup guide? i'm not sure what it is. |
hi! @killthe3vil, i should have asked if you did anything differently when it didn't work and for how long. (the update(v4.443) was only approved for our chrome webstore beta testing version yet)
the repo is also the extension. no building required. |
Hi @ImprovedTube |
Hey @killthe3vil its working on my side too |
@shadow-ryu ya it weird that it fixed itself by today |
we can keep this issue on monitoring? |
it is me only or the function not working again? |
hi @killthe3vil thanks, we can just assuming it works for most people most of the time and we just had a bug with applying it to embedded youtube players before. ( assumably you werent confused by the music exclusion either. )
(didnt mean to say there is no bug, just that we should fix it already if any.) yes, looking forward to whatever we might discuss next. |
tried reinstall? |
Yes, I reinstalled and restored settings and still got 1.25 on music. |
@xeophyte how many examples did you try? please share all or look for the reason they were missed
|
On at least 15 videos. Actually the 1.25 speed often doesn't work even on talking movies, like https://www.youtube.com/watch?v=lRvXAU9_sfc or https://www.youtube.com/watch?v=DKBxkrUPCbs even after many refreshes. Sometimes it works, but then music doesn't, like I play music video, get 1.0, then go directly to talking, get 1.25, then go back to music and 1.25 stays. |
@xeophyte so it worked better for you before? (v4.408, v4.404, ... ?)
with browser nagivation back arrow? maybe this code doesnt always run for you when a video starts? do all these other features, such as the permanent volume, also work - or fail at the same times? youtube/js&css/web-accessible/functions.js Lines 344 to 365 in 7e2519a
|
@xeophyte
to confirm this please check if everytime it fails, the browser console sais 5 times: Error: Invidious API: ... and eventually (~10s later) Error parsing id="scriptTag"\'s JSON: ..... |
I test this feature only for few days, don't know how it was earlier.
Yes, and also when I click music video directly from the right related list. Similar test: I play talking video (sets itself on 1.25), go to music from related (stays at 1.25), set it to 1.75, go to another talking (sets itself on 1.25), go to some music (stays at 1.25), set it to 2.0, go to some music (stays at 2.0). It doesn't matter if the next video is opened by click or using the previous/next page button, the effect is the same. Maybe it's somehow related to YT itself remembering the speed setting of a given video?
I don't know how to test it.
|
thank you! @xeophyte you can watch the browser console live to see what happens when. (busy seconds. Can also enable top view timestamps..)
|
I found some videos get the automatic speed and some do not. I am guessing this has to do with the category but I can't find where to see what categories a video belongs to |
@mrlinuxfish yup that how the situation to me as well, could not figure out what the issue |
thanks! @mrlinuxfish @killthe3vil maybe it happens only when excluding music and only after clicking a related video only?
hoped we'd have an easy bug, that always excludes education videos as the switch is new, but maybe not you can see the category called genre in the browser console log as in the screenshot above our comments |
let us know your further experience? @mrlinuxfish @killthe3vil @shadow-ryu @xeophyte |
i think so far so good now, but sometime will not working like 10 out of 1 time. |
Looks like it's working for me on 4.700 (maybe one out of 10 times it doesn't work), although sometimes switching to the appropriate speed takes up to 10 seconds. |
Try open video in a new tab and wait that 10 second before tuning into it. It should work from a strat. Some things need some time to load. |
It doesn't make sense when the playlist is playing, after all, that's why it's there so that you don't have to open every video in a new tab. |
Option to control speed on playlist or exclude playlist from other vieos could be cool option. Remember @ImprovedTube WE DON JUST HAVE PLAYLISTS, there are queues, mixes and probably something more. |
@ImprovedTube |
hi! @xeophyte yes, when music only is indicated as in youtube's catalogue /buyable, below the video-descriptions. How often? yes, inversely (and originally for us), these few music videos can start slow instead. But the feature is used for speed-watching mostly and the music sub-option enabled by default for this feature. So @Eyevou thought of the current order And maybe hurried users like it like now. As of the edit history anyone and chatgpt (next to me) can can now add a sub-sub-option to allow both detection orders. ( long before/easier before ultimately one day our music detection should even be immediate always, if ever. ) |
I didn't understand any of it. I just want to play music videos at 1.0 when the Playback speed is set to 1.25. Now it starts playing at 1.25 or it is set like that all the time. |
really? @xeophyte:
And here you suggest to not undo speeding up already in the first second before noticing music.. - This order was only introduced in January as per @Eyevou's suggestion and can be undone or we can easily-enough have both options. you will succeed soon if you try. |
this youtube/js&css/web-accessible/www.youtube.com/player.js Lines 82 to 204 in 85bf124
really needs refactoring, reformatting and cleaning up |
it been few days the playback speed not working anymore.
The text was updated successfully, but these errors were encountered: