-
-
Notifications
You must be signed in to change notification settings - Fork 123
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
Issue with SponserBlock / What is blocked. #362
Comments
Thanks for the issue, I'll look into it. |
Sounds good! Good luck trying to resolve this. |
This is an issue for me too, sponsorblock also does not seem to do anything. |
I managed to hack it back into working order by manually editing the youtube.py and copy pasting from a code snippet in a reddit post, I don't know enough about python to submit a PR unfortunately. 'postprocessors': [
{'key': 'SponsorBlock'},
{'key': 'ModifyChapters', 'remove_sponsor_segments': ['sponsor', 'intro', 'outro', 'selfpromo', 'preview', 'filler', 'interaction']}
] Removing https://github.com/meeb/tubesync/blob/main/tubesync/sync/youtube.py#L117 also got rid of the error I mentioned in my previous post. (I appended ModifyChapters below it.) (Not completely sure if this would also solve the settings disappearing) |
Your patch is pretty much exactly what the default sponsor block management code should do, selecting the tick boxes should just generate the list like There's a bug somewhere in the handling and creation of the the list that only affects some deployments so it's tricky to catch. |
fyi im getting this issue also. I can give details of my setup /the yt channel if you think it'd help with your debugging/ resolution of this. thanks. |
I am using this in unraid do you know if there is an option to make that change to the system? I have been trying but not successful. |
@Lucas319315 that edit would be made in the https://github.com/meeb/tubesync/blob/main/tubesync/tubesync/settings.py#L157 |
Thanks for the assist! |
Is it possible we're able to have an incremental release with this fix in please so we're not all having to manually edit files? That would be great. |
There isn't a fix in the repo for this yet @mylogon341 - I've not been able to replicate the issue to debug it. If anyone has a bare SQLite database from a tubesync install which exhibits this behaviour feel free to add it to the issue. |
Sorry - ignore. I got myself mixed up on bugs I'm following and just added a comment in my 'rona recovery. In my head this was for a ticket for sponsor block not working properly. Maybe this is related and maybe not 🤷♂️ I'll maybe come back to this and do some more testing when my brain-fog clears. |
I'm pretty confused about how to edit that settings.py file. I'd really like to get SponsorBlock to work. |
@duloo0 you should just be able to tick the sponsorblock categories from the edit source form, unless you are impacted with this bug. |
I'm impacted by the same bug unfortunately. |
How big is your SQLite database? If you can upload it so I can replicate the issue I can fix why the option isn't working. |
Hi, I'm having this issue, or at least a similar one where sponsorblock isn't working. I have Sponsorblock enabled with:
And I get:
in my logs when I try to redownload a video. The key thing seems to be the Also happens with just the "sponsor" option checked along with "enable sponsorblock". As far as I can tell, digging around , we just need to change:
to
But that's just my impression from looking at the code and is entirely untested. I'll try to play with it and possible post a PR sometime over the next week or so. My DB is like a gig, so it probably isn't feasible to just upload. If I can't confirm or figure out a fix, I'll spin up an instance with a pared down version of the problem and post the db |
@meeb Okay, I was able to reproduce this on main (currently 6853c1f) with a small database which I've attached. tubesync-sponsorblock-broken-db.zip To reproduce the
I'll continue looking at this, but I'm less confident that my proposed solution will work. |
Thanks @thavelick - I've pushed a very minor tweak that may help with this. It'll be in the next release and |
@meeb I built a container with the above commit and now I'm getting:
Seems like we're getting closer! But sponsor segments are still in there |
Ah, well that looks like TubeSync's end is working, I just need to find why the upstream API is blocking the requests then (or find an alternative endpoint). Edit: or perhaps the category names are wrong and that's what the API is complaining about? Mostly a note for future me to look into. |
Yep it was me being an idiot and wrapping a list in another list as a typo. This should be fixed once the above commit builds into |
Can you please commit the builds into latest so I can pull the latest image, ads in my videos are killing me. Thank you ! |
@irishj this should be in |
Thanks @meeb - I'll pull the latest and check it out. Thanks again for this great software. |
I'm having the same issue here, but I don't think those commits are helping. The problem seems to be that it clears the sponsorblock settings whenever it fetches the metadata. Versions tested: 0.13.3 and master
I tried digging into it a bit more and it looked like it was the bit in the |
Thanks for investigating! I'll take a look at your reproduction steps. |
I'm seeing a similar issue with existing sources which have already been added. I can set the sponsor block options via the GUI or via a MySQL query (I'm using MySQL as the back-end DB) and the chosen block options are saved, but when the scheduled tasks are run, the sponsor block options are removed for all sources. The sponsor block enabled option is still enabled, but the sponsor block options for each source have all been removed. |
Yeah there's clearly a bug in the handling of the sponsorblock categories. I'll look into it when I get time. |
Would love to see this fixed, it's the last thing in my archiving server to sort out. I'm not brave enough to edit the containers I'm running locally, but I'm happy to test out any pushed container updates that get made to address this bug. |
I've had a couple of looks at this and it's only not fixed because I've never seen it happen myself, no-one has been able to supply a trace of an error or a copy of an SQLite database that exhibits this behaviour so it's obviously quite hard to locate the bug. The reproduction steps haven't reproduced it for me in testing. If you can supply your SQLite database somewhere, if you're OK with sharing that, I could probably locate the issue. |
tubesync-sponsorblock-repro.zip I just made this compose file, ran it, added a non-existent channel as per uduncanu's steps, ticked the box for "(ALL OPTIONS)", saved the source with an index schedule of 7 days, and waited. The page we get redirected to shows that sponsorblock settings were saved, but refreshing it (roughly when indexing has started) shows that the settings disappear. I'm using Docker Desktop on Windows 11, hence the funky Docker Compose version used in the file too. It doesn't seem to matter on my end, but my other containers are using a PostgreSQL database instead of SQLite. Sponsorblock issue happens either way. |
Excellent, thanks. I'll see if I get some time later this week to spin up a test instance with your database. |
Hello Again. Just wondering if you had an opportunity to review this issue / find a solution to sponsor block not working ? |
This is working for me. |
Nothing has updated in the repo with regards to this issue, why would you close this? If your manual edits work, @Lucas319315, then that's great, but they need to be added to this repo. |
Yeah the manual edits worked for me. I am new to github and don't know the best process on leaving this opened or closed. |
@Lucas319315 Can you please share a copy of your "settings.py" file where you have made the changes so I can try it ? I made the edits myself manually in the "settings.py" file and enabled the sponsor block categories for one channel, ran the "reset tasks" script and after the channel was indexed, the sponsor block categories were reset / disabled again. Maybe I made the changes incorrectly. Thanks.
|
@irishj I do know meeb mentioned the following below but that only worked for about a day and then it would not work anymore where the edit to youtube.py I did has not broken since but it does need be be reapplyed after any updates to that docker and it does apply those settings across all sources. The change is on line 147 |
Thanks for sharing your file. I tried it within my own installation and within a new installation and after a channel is synced, the sponsor block options are being disabled, not sure why that might be. Will probably need to wait for @meeb to sort out the issue so I can finally get rid of all those annoying ads. |
I check my sources menu and when I go into it shows sponser block enabled but the "what blocked?" is empty.
I go and re-add the categories again but when I eventually go back to it that section is blank again.
The text was updated successfully, but these errors were encountered: