Cheesegeezer 3086 Posted December 9, 2021 Share Posted December 9, 2021 12 minutes ago, chef said: Then we should put a hard limit at 4. @Cheesegeezer, @rbjtech It makes sense to have quality over quantity... I think. If a lower scan rate is proving more successful, then should we cap the threading? We would still have the xml config, for extreme cases, could be edited to increase the detection rate manually. However, the UI should cap at 4. I don't think a UI hard limit. Should be installed. I can't remember what our default is (2 or 4). We do have a disclaimer that says low power machines should set to 2. However, if you are like @rbjtech with a fancy new 12700 processor, then those would like to be able to up the limit without editing xml. we could set a hard limit of 4 with a toggle switch to "increase default Values" if switched a popup dialog can then have information stating some known scenarios and the effects it could have on the scan, with a continue and cancel button. In the future processors will get faster and cheaper people will upgrade and the limit of 4 will be redundant as the new limit will be 8 for example. 9 minutes ago, Painkiller8818 said: But i have "only" 8 physical cores, maybe someone has a server with 24 cores. But yes, i have much better results with 4 sessions Do we know if its the detection that causes the issues having this set to 8 or if it was the FP process that didn't write the Chroma data well enough. I will do a check on the Chroma values. If they are identical with it set to 2 parallel and 8 parallel then we can see that it's the detection that falls down when it hits the cpu limitiations. There are too many variables to content with in terms of hardware configurations. Sorry that was my brain dumping out there. 2 Link to comment Share on other sites More sharing options...
Painkiller8818 203 Posted December 9, 2021 Share Posted December 9, 2021 2 minutes ago, Cheesegeezer said: Do we know if its the detection that causes the issues having this set to 8 or if it was the FP process that didn't write the Chroma data well enough. I uploaded a log but haven't got any response on it. i watched both NAS and neither CPU nor DISK usage was high (this is where my movies are) If the Emby Servers local Disk is used for this, there is absolutely nothing, i have a very fast m.2 Disk with read/write speed of about 4-6K MB/s Link to comment Share on other sites More sharing options...
chef 3745 Posted December 9, 2021 Share Posted December 9, 2021 33 minutes ago, Cheesegeezer said: I don't think a UI hard limit. Should be installed. I can't remember what our default is (2 or 4). We do have a disclaimer that says low power machines should set to 2. However, if you are like @rbjtech with a fancy new 12700 processor, then those would like to be able to up the limit without editing xml. we could set a hard limit of 4 with a toggle switch to "increase default Values" if switched a popup dialog can then have information stating some known scenarios and the effects it could have on the scan, with a continue and cancel button. In the future processors will get faster and cheaper people will upgrade and the limit of 4 will be redundant as the new limit will be 8 for example. Do we know if its the detection that causes the issues having this set to 8 or if it was the FP process that didn't write the Chroma data well enough. I will do a check on the Chroma values. If they are identical with it set to 2 parallel and 8 parallel then we can see that it's the detection that falls down when it hits the cpu limitiations. There are too many variables to content with in terms of hardware configurations. Sorry that was my brain dumping out there. This all makes sense. We could look at environment processors, and have a popup if the user increases the value beyond the cores / 2. Or the toggle to increase the value beyond recommend values. "Ludacris speed" "going plaid"... Anyone?... Spaceballs? Link to comment Share on other sites More sharing options...
chef 3745 Posted December 9, 2021 Share Posted December 9, 2021 (edited) I did this: Those two settings are locked by default: You can unlock them, then edit the values: But, if you lock the setting again, the settings go back to default (value of 2). I was able to use the value of 6 on my server, and get great results. I think that in most cases, when users unlock the setting, we should recommend 6, unless they have a massive server that can handle 8 or 16. Low power machines should just leave the setting locked, which is a default of 2. I don't know if this will help, but, at least there is a toggle that will reset to defaults if things are not working for people. Edited December 9, 2021 by chef 1 Link to comment Share on other sites More sharing options...
rbjtech 4262 Posted December 9, 2021 Share Posted December 9, 2021 The two tasks have different hardware dependencies - FP is disk I/O, Detect is CPU with a little disk I/O for the end credit. It's impossible for us to cater for every scenario - I think we just need to have text with the above info, and let the user make the choice but keep the defaults low (as they are now). imo. 1 Link to comment Share on other sites More sharing options...
fallenwitch3r 41 Posted December 9, 2021 Share Posted December 9, 2021 (edited) @chefi have the same problem with the chapter insertation again. I have add a new series and all intros are correct identified but the chapter insertation did not work. Even refresh metadata has not worked. The intro starts at 00:00 and end at 01:44 on almost all episodes. Edited December 9, 2021 by fallenwitch3r Link to comment Share on other sites More sharing options...
chef 3745 Posted December 9, 2021 Share Posted December 9, 2021 1 minute ago, fallenwitch3r said: @chefi have the same problem with the chapter insertation again. I have add a new series and all intros are correct identified but the chapter insertation did not work. Even refresh metadata has not worked. @Cheesegeezer is the chapter guy. He knows that part of the code the best. We might need to see a log. Link to comment Share on other sites More sharing options...
fallenwitch3r 41 Posted December 9, 2021 Share Posted December 9, 2021 (edited) 27 minutes ago, chef said: @Cheesegeezer is the chapter guy. He knows that part of the code the best. We might need to see a log. @Cheesegeezer @chef send you a pm Edited December 9, 2021 by fallenwitch3r 1 Link to comment Share on other sites More sharing options...
benwallner 2 Posted December 9, 2021 Share Posted December 9, 2021 On 08/12/2021 at 18:32, rbjtech said: Blame Google Translate - that's where I got the translations from ! But thanks, a human touch always provides a much nicer translation A little tip for the future try to use DeepL it has the correct translation https://www.deepl.com/translator#en/de/skip intro 2 Link to comment Share on other sites More sharing options...
Cheesegeezer 3086 Posted December 9, 2021 Share Posted December 9, 2021 2 hours ago, fallenwitch3r said: @chefi have the same problem with the chapter insertation again. I have add a new series and all intros are correct identified but the chapter insertation did not work. Even refresh metadata has not worked. The intro starts at 00:00 and end at 01:44 on almost all episodes. Can you check the Chapters Tab and see if you have chapter errors pop up please. Thanks Link to comment Share on other sites More sharing options...
fallenwitch3r 41 Posted December 9, 2021 Share Posted December 9, 2021 (edited) I added season folder by season folder and not the whole folder which includes all episodes and maybe this is the error. I cleaned up the folder from all meta files after moving the folder to the main dir. I let the system do a rescan on the series folder. Then I copied the folder back and after adding all metadata by the system the intro skip plugin did his work like before. All timestamps and chapters are now correctly generated. Edited December 9, 2021 by fallenwitch3r Link to comment Share on other sites More sharing options...
chef 3745 Posted December 9, 2021 Share Posted December 9, 2021 I havent read all the chapter code. Interesting that is was picked up after a complete refresh. Link to comment Share on other sites More sharing options...
Cheesegeezer 3086 Posted December 10, 2021 Share Posted December 10, 2021 2 hours ago, chef said: I havent read all the chapter code. Interesting that is was picked up after a complete refresh. There isn't much to it chef. just lots of organising the chapter positions. There was nothing in the logs either. 1 Link to comment Share on other sites More sharing options...
fallenwitch3r 41 Posted December 10, 2021 Share Posted December 10, 2021 Is that bug which causes the image loading on the chapter section fixed? Or are there any news about an upcoming fix? I went back to the previous beta after the bug gets public. Link to comment Share on other sites More sharing options...
Cheesegeezer 3086 Posted December 10, 2021 Share Posted December 10, 2021 30 minutes ago, fallenwitch3r said: Is that bug which causes the image loading on the chapter section fixed? Or are there any news about an upcoming fix? I went back to the previous beta after the bug gets public. I asked for more information to look at it, however you have not replied to the PM or on here. You just went ahead and found a work around and now I have no idea what caused the issue in the first place. 1 Link to comment Share on other sites More sharing options...
fallenwitch3r 41 Posted December 10, 2021 Share Posted December 10, 2021 17 hours ago, fallenwitch3r said: I added season folder by season folder and not the whole folder which includes all episodes and maybe this is the error. I cleaned up the folder from all meta files after moving the folder to the main dir. I let the system do a rescan on the series folder. Then I copied the folder back and after adding all metadata by the system the intro skip plugin did his work like before. All timestamps and chapters are now correctly generated. This is what I did. I don't know why or how it worked. Link to comment Share on other sites More sharing options...
chef 3745 Posted December 10, 2021 Share Posted December 10, 2021 Hey! I have credit sequences skipping While doing so, I found a bug. I'm just going to make sure I can track it back to where the problem starts. Then I can post a dll. 4 Link to comment Share on other sites More sharing options...
chef 3745 Posted December 10, 2021 Share Posted December 10, 2021 It's close to a release, we just have to fix a couple things, and build a language dictionary. 4 2 Link to comment Share on other sites More sharing options...
blackstar88 2 Posted December 19, 2021 Share Posted December 19, 2021 On 12/10/2021 at 9:07 PM, chef said: It's close to a release, we just have to fix a couple things, and build a language dictionary. Pm me if u need swedish translation 1 Link to comment Share on other sites More sharing options...
magicgg91 2 Posted December 24, 2021 Share Posted December 24, 2021 Hello, Would you think the possibility to improve the exclusion capability ? I have almost 150 series, and I wanted to try this plugin with 2 or 3 to get an idea on how it works and see what are the limits I can set. But as it's actually available, it's a pain to add one by one series to exclusion list. It should be good to have a button to add a full library to exclusion list. And when I want to remove some series from exclusion, the list is cropped, I can't see full list, but only around 50 series. I think it's only a display bug, because if I search with browser for one serie that I konw excluded, it found 1 result, but can't display it, it's too low on the page. Thanks for this great plugin ! I wish all of you my best wishes for Christmas ! 2 Link to comment Share on other sites More sharing options...
fbrassin 35 Posted December 27, 2021 Share Posted December 27, 2021 Hello, is it all ok with this project? I ask because it has been 2 weeks with no replays, after a period full of them. Link to comment Share on other sites More sharing options...
henryford 11 Posted December 27, 2021 Share Posted December 27, 2021 It's holiday season, give the devs some slack 1 Link to comment Share on other sites More sharing options...
chef 3745 Posted December 27, 2021 Share Posted December 27, 2021 On 12/24/2021 at 1:45 PM, magicgg91 said: Hello, Would you think the possibility to improve the exclusion capability ? I have almost 150 series, and I wanted to try this plugin with 2 or 3 to get an idea on how it works and see what are the limits I can set. But as it's actually available, it's a pain to add one by one series to exclusion list. It should be good to have a button to add a full library to exclusion list. And when I want to remove some series from exclusion, the list is cropped, I can't see full list, but only around 50 series. I think it's only a display bug, because if I search with browser for one serie that I konw excluded, it found 1 result, but can't display it, it's too low on the page. Thanks for this great plugin ! I wish all of you my best wishes for Christmas ! We have a release that allows you to add all the items to the exclude list, then you can remove the ones you want to scan. Link to comment Share on other sites More sharing options...
chef 3745 Posted December 27, 2021 Share Posted December 27, 2021 (edited) We are finishing a wiki over on GitHub. https://github.com/chefbennyj1/Emby.IntroSkip.Release/wiki/Emby-Intro---Credit-Skip Here is the latest beta build of the plugin. It is not perfect. It has come a long way. There are a couple known UI issues.... but I think overall it is a beta worth checking out. Please clear your browsing data before navigating to the configuration page. IntroSkip_v0.0.1.55.zip Edited December 27, 2021 by chef 2 1 Link to comment Share on other sites More sharing options...
chef 3745 Posted December 27, 2021 Share Posted December 27, 2021 (edited) Initial notes: Please don't decompile our project for the sake of using the code in other projects. We have many open source emby plugin projects on github to read and borrow from. V.0.0.1.55 release notes: Auto skip (you'll have to check it out to see what it is... there is a lot of features there). Add all series to exclude list Configuration UI option to cache images to the file system (beware: image cache can get very very big, very quickly) Configuration UI caches images locally in the browser for speedy loading after the UI loads a season the first time. (different from the file system cache). Local browser Cache is globally scoped, so as long as the emby dashboard is open, you can browse back to the intro skip configuration and the cache will persist for fast loading. Setting a "true" detection to "false", will reset the Extracted Thumb image, and set it's timestamp to "00:00:00". Don't forget to "Save". Pagination added to the Sequence table, so we will only load (and attempt thumbnail extractions) for five sequences at a time. This helps loading times, as well as limits CPU usage while editing sequence data. Known limitations: Linux users on Emby beta release can not see extracted thumbnail images for sequences (problem with linux ffmpeg on beta release) spinning circle does not disappear after season reset (fixable) Localization is not being used for credit skip Some localization strings need updating for intro skip message. Sending a message during intro causes some websocket issues if the client reports the capability to receive messages but actually can not. Emby API says we can control the message duration (API: "SendMessage") I don't believe this is true. Option in the intro skip plugin for message duration may not actually affect the duration. Edited December 27, 2021 by chef 4 1 Link to comment Share on other sites More sharing options...
Recommended Posts