reggi 409 Posted March 30, 2017 Author Share Posted March 30, 2017 They should be all sortable. But I changed the script and forgot to place the column names in a <span> tag. In the next update tonight they will be sortable again Thanks for mentioning!! As for the > 100% collected stats. I still have to check why it is happening. I already found out that with some episodes TVDB has no aired date (but they are already aired) 1 Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted March 30, 2017 Share Posted March 30, 2017 I have just checked Babylon 5 which shows as 96.5% On TVDB.com B5 has 110 episodes + 8 Specials - i have all of these files i also have 19 other specials not listed on TVDB the Plugin is showing 110/114(96.5%) +27sp so the 27sp is correct what i cant figure is where the 114 is coming from? Link to comment Share on other sites More sharing options...
reggi 409 Posted March 30, 2017 Author Share Posted March 30, 2017 @@PenkethBoy Did you check the content of your Babylon 5 folder? If you can, can you send me a PM with the content of that folder so I can try to figure out why he is counting 4 more episodes? Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted March 30, 2017 Share Posted March 30, 2017 you already have the listing i sent the other week files have not changed and yes there are only 110 + 27 files in the directories Link to comment Share on other sites More sharing options...
reggi 409 Posted March 30, 2017 Author Share Posted March 30, 2017 Ha ok thanks! if they are not changed I will have a look at that listing. 1 Link to comment Share on other sites More sharing options...
reggi 409 Posted March 31, 2017 Author Share Posted March 31, 2017 Here is a new version that will fix the double episode issue. As for the strange stats for Babylon 5 @@PenkethBoy has, I think this will be fixed in this version as well. Let me know if the total episode count is still wrong. Statistics 0.0.11.4.zip Link to comment Share on other sites More sharing options...
Happy2Play 8282 Posted March 31, 2017 Share Posted March 31, 2017 (edited) Looks like the multi-episodes are doubled. Just a quick check since this is the easiest to find, will try to look for more examples. Mob City Grimm has one multi-episode and I have 121 files so 122 episodes and Stats shows 142 / 142. Edited March 31, 2017 by Happy2Play Link to comment Share on other sites More sharing options...
reggi 409 Posted March 31, 2017 Author Share Posted March 31, 2017 Why is this not happening on my server I'll fix it when I get home this evening! Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted March 31, 2017 Share Posted March 31, 2017 Hi Reggi just updated to 11.4 and the B5 numbers are the same 110/114 the 114 i assume is coming from TVDB via the api - do you get data from anywhere else? could the api data be different to what i see via the "web page" for the series? You have found one error for me though - my last season of Defiance had a double episode which i had missed - so the show was showing as one missing Link to comment Share on other sites More sharing options...
reggi 409 Posted March 31, 2017 Author Share Posted March 31, 2017 I will check the xml file returned from the TVDB api, but normally this should be the same as the web page data. Probably it returns 114 episodes in the xml but I'm just counting them wrong. Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted March 31, 2017 Share Posted March 31, 2017 To aide you in your search for the error I also have "The West Wing" with 154/155 and "Yes Minister" 21/22 i have all the episodes 154 and 21 as per TVDB Rest appear correct Link to comment Share on other sites More sharing options...
reggi 409 Posted March 31, 2017 Author Share Posted March 31, 2017 Hey guys, I fixed some more issues. For the total episode count that was too high (@PenkethBoy) I figured out I was using the wrong xml tag to extract the season number. So I fixed that, added "Yes Minister" to my lib and I got 21 episodes for total count now! For the problem @@Happy2Play is having (double episodes counted twice), I forgot to add parenthesis around a null check //Wrong code, if IndexNumberEnd is NULL it will return (r.IndexNumber - r.IndexNumber + 1) distinctList.Sum(r => r.IndexNumberEnd ?? r.IndexNumber - r.IndexNumber + 1) //Correct code distinctList.Sum(r => (r.IndexNumberEnd ?? r.IndexNumber) - r.IndexNumber + 1) So if you guys could test it again it would be a lot of help! If you install the 0.0.11.5 version remove the xml file before running the task! Because I needed to change the xml tag and all data needs to be refreshed. I forgot to mention, I added a "total collections" stat on the general stat page in the previous version. Statistics 0.0.11.5.zip Link to comment Share on other sites More sharing options...
Happy2Play 8282 Posted March 31, 2017 Share Posted March 31, 2017 Looks like that did it for multi-episodes. Still something wonky with shows that air today/tonight show already "Collected". Won't have the episode til later tonight. examples Grimm 2011 122 / 123 (99.2%) 123 / 123 (100%) MacGyver 2016 18 / 19 (94.7%) 19 / 19 (100%) Hawaii Five-0 2010 162 / 163 (99.4%) 163 / 163 (100%) 1 Link to comment Share on other sites More sharing options...
reggi 409 Posted March 31, 2017 Author Share Posted March 31, 2017 Happy to hear the multi-episode problem is sorted! As for the new problem, I added Hawaii Five-0 to my lib and added 1 dummy episode. My stats results just show 1 collected episode. So on my server, I don't have issues with shows that are going to be aired today/tonight. I'll have to investigate some more tomorrow!! Thanks for the response. Link to comment Share on other sites More sharing options...
Diedrich 355 Posted April 1, 2017 Share Posted April 1, 2017 Another submission for the artwork thumb. This one is a bit more conservative and mainstream. The border is just to define it from the white forum background. Attached is the png file. Here is the AI file 2 Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted April 1, 2017 Share Posted April 1, 2017 @@reggi that update fixed B5, Yes Minister and TWW all counting correctly now was just looking at the first screen and possibly could do with a slight change as you can see - either word rap or split age and name over two lines also with Newest added episode - could do with TV series name Link to comment Share on other sites More sharing options...
reggi 409 Posted April 1, 2017 Author Share Posted April 1, 2017 @ Thanks! sorry I forgot to reply on your first idea! But this looks better indeed (clean and simple). I like it!! @@PenkethBoy Glad to hear it is fixed now, I'll see what I can do for the layout today as well. 2 Link to comment Share on other sites More sharing options...
Animosity022 6 Posted April 1, 2017 Share Posted April 1, 2017 Is there a way to compile this up on Debian/Linux? I did some searching and didn't have much luck. Link to comment Share on other sites More sharing options...
reggi 409 Posted April 1, 2017 Author Share Posted April 1, 2017 You don't have to, just copy the dll into the Emby server plugin folder on your machine. On Ubuntu it is "/var/lib/emby-server/plugins", don't know for sure where it is on Debian. Link to comment Share on other sites More sharing options...
Animosity022 6 Posted April 1, 2017 Share Posted April 1, 2017 Thanks. I didn't read through the whole thread and I had downloaded from github and was trying that route. Same spot on Debian as well Thanks again Link to comment Share on other sites More sharing options...
reggi 409 Posted April 1, 2017 Author Share Posted April 1, 2017 Glad it's working now! I'll maybe add the info for Linux users on github. if you have any ideas or see any bugs in the plugin let me know. I'm trying to get it ready for the catalog. Link to comment Share on other sites More sharing options...
Diedrich 355 Posted April 1, 2017 Share Posted April 1, 2017 (edited) Reggi, you've done a great job, man! It looks great and I don't see any issues on my end. Fine job. I look forward to seeing it out in the wild and the reaction from other users. Edited April 1, 2017 by dvgmar Link to comment Share on other sites More sharing options...
reggi 409 Posted April 2, 2017 Author Share Posted April 2, 2017 Hey guys, Here is a new small update 0.0.11.6 I changed the stats so everything is split over 2 lines (if needed) Also in the user based stats I added 3 more stats - Total Movies Watched - Total Episodes Watched - Total Series Finished Besides @@Happy2Play last mentioned problem (Still something wonky with shows that air today/tonight show already "Collected". Won't have the episode til later tonight.) I think this could be the first RC version. Statistics 0.0.11.6.zip Link to comment Share on other sites More sharing options...
PenkethBoy 2063 Posted April 2, 2017 Share Posted April 2, 2017 (edited) Hi Reggi with 11.6 first second - the table for shows is not sorted when first displayed rest ok Questions 1) Are you going to do a simplified table for movies ( without the lookup) showing basic info and number of "extras" 2) Are you going to do something for Unset, home videos, music etc - ie the other library types? Edited April 2, 2017 by PenkethBoy Link to comment Share on other sites More sharing options...
reggi 409 Posted April 2, 2017 Author Share Posted April 2, 2017 @@PenkethBoy Thanks for the review, I fixed the sorting issue and I'll have a look at the "12 months" thing now As for your requests, I will add them in github so I can keep track of what to implement next. The simplified movie table sounds like a good idea, as for the other library types I think I will first get the movies and shows in order before implementing the other types. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now