Is Comskip meant to work in live tv?

Discussion in 'Television (MyTV frontend and TV-Server)' started by erosco, July 9, 2016.

Tags:
Page 1 of 2
  1. erosco
    • Premium Supporter

    erosco MP Donator

    Joined:
    June 25, 2007
    Messages:
    122
    Likes Received:
    10
    Ratings:
    +11 / 0
    Home Country:
    July 9, 2016 #1
    According to http://wiki.team-mediaportal.com/1_MEDIAPORTAL_1/0_What's_New/1.9.x/Live_Commercial_Skip MP is supposed to pick up and monitor an .edl file creating during recording and make it available during playback.

    Whilst I have 100% success playing back recordings and seeing the inserted chapter marks AFTER MP has finished the recording.. However If I start watching the recording at any time during the recording no chapter marks are visible - even when I start playback at the last few seconds of the recording... I have confirmed that the .edl file is being created in real-time and updated with chapter marks in real-time.



    Is this meant to work or has the code been pulled? - I have tried an RTSP client, UNC paths and local paths all without any success.

    Many Thanks
    erosco
     
    erosco, July 9, 2016

  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,448
    Likes Received:
    4,663
    Ratings:
    +8,100 / 17
    Home Country:
    New Zealand New Zealand
    July 9, 2016 #2
    Hi

    In other words, you followed the instructions in the linked wiki page to update the Comskip "live_tv" and "output_edl" config, and you configured the TV Server ComSkip launcher plugin to launch ComSkip at the start of the recording...

    ...right? (Please confirm.)

    It definitely hasn't been pulled.

    I don't think Comskip marking works with live TV (though I'm not 100%).
    Assuming Comskip is enabled, configured correctly and has had enough time to do its processing, it should work with complete recordings... and my understanding is that it should also work with in-progress recordings.

    Important
    For it to work with in-progress recordings, you must choose to watch the recording from the TV plugin's recording list. Viewing the same channel via live TV while a recording is in progress is not the same thing and will not work (assuming I'm right that Comskip marking doesn't work with live TV).
     
    mm1352000, July 9, 2016
  4. erosco
    • Premium Supporter

    erosco MP Donator

    Joined:
    June 25, 2007
    Messages:
    122
    Likes Received:
    10
    Ratings:
    +11 / 0
    Home Country:
    July 9, 2016 #3
    Hi - Yes, I followed all the steps live_tv and output_edl. Comskip is started at the start of the recording. I have my recordings directory shared so I can monitor the edl over time.. I see the commercial marks in seconds being added to the file whilst the program is being aired (comskip in live mode keeps up with the recording).

    I am also starting the recording from the 'mytv' 'recordings' directory - I assume this is what you meant.

    My only difference is that I'm not using comskip_launcher in the tv server due to a bug with it! (quotes in the parameters fails to launch a .bat file - which is needed for per channel .ini file selection). I use opensource "File Watcher Utilities" that launch comskip as soon as the .ts file is created.. - Could this be my problem? Everything still happens at the right times.. just not started by comskip_launcher.


    thanks
    erosco
     
    erosco, July 9, 2016
  5. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,448
    Likes Received:
    4,663
    Ratings:
    +8,100 / 17
    Home Country:
    New Zealand New Zealand
    July 9, 2016 #4
    Not sure, but I doubt it.

    Can you provide Watchdog log files showing the situation?
     
    mm1352000, July 9, 2016
  6. erosco
    • Premium Supporter

    erosco MP Donator

    Joined:
    June 25, 2007
    Messages:
    122
    Likes Received:
    10
    Ratings:
    +11 / 0
    Home Country:
    July 9, 2016 #5
    Thanks

    Will do - sorry - never used the watchdog; might take me a bit to work out how :)
    --- Post updated ---
    As mediaportal doesn't abort I have attached the logs after I have started the recording, waited 10mins for an ad break; confirmed an .edl - attached; started to view the program via the recordings menu in mytv - seen no ad breaks in the osd menu (no red lines).

    Seems I can't upload the .edl - not allowed. changed file to .edl.txt

    Thanks
    erosco
    --- Post updated ---
    Ok - Seems I have an apology to make. I think I managed to get it to sort of work.

    I killed off comskip half way through the show and then watched the recording (whilst still recording in mediaportal) and the chapter marks that had been identified in comskip appeared.

    I am wondering if comskip now had an 'exclusive write' flag or something on the file - seems this might be a(nother) comskip issue and not mediaportal..

    I am very sorry for wasting your time.. now I need to work out why.

    (EDIT) - Maybe if comskip is started by the plugin - the files are accessible by TVService as it's the parent process - ?!!? - this might be the issue!

    MM1352000 - Many thanks for your help.

    Best Regards
    erosco
     

    Attached Files:

    erosco, July 9, 2016
  7. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,448
    Likes Received:
    4,663
    Ratings:
    +8,100 / 17
    Home Country:
    New Zealand New Zealand
    July 9, 2016 #6
    Hello again

    Sorry for the delayed response - I got waylaid by other priorities.

    No need to apologise to me. :)

    Yeah, that process should be fine. (y)
    The Watchdog just collects log files when MP is closed. That closing can be a crash or a standard "user-initiated" close.

    The MediaPortal log file you provided shows:
    [2016-07-09 10:23:03,005] [Log ] [MPMain ] [DEBUG] - g_Player.LoadChapters() - Automatic ComSkip mode is on.
    [2016-07-09 10:23:03,006] [Log ] [MPMain ] [WARN ] - g_Player.LoadChapters() - Invalid chapter file

    So, the chapter file is seen but not loaded, and that explains why you don't see the markers.

    I think the relevant code is here:
    https://github.com/MediaPortal/Medi...diaportal/Core/Player/g_player.cs#L3438-L3443
    Code (Text):
    1.  
    2.   int fps;
    3.   if (String.IsNullOrEmpty(line) || !int.TryParse(line.Substring(line.LastIndexOf(' ') + 1), out fps))
    4.   {
    5.   Log.Warn("g_Player.LoadChapters() - Invalid chapter file");
    6.   return false;
    7.   }
    In other words, it looks like MP is not picking up the Comskip "chapters" because it fails to read the frames-per-second detail from the end of the first line of the Comskip file.

    Looking at the contents of the EDL file, I can see that the end of the line has a tab followed by a zero. MP is expecting a space instead of a tab... and I suspect a real FPS value is needed too.

    Perhaps there are relevant Comskip options - relating to file format and/or frames per second - which you're aware of?
    (I've never used Comskip, so this is where my ability to help you becomes limited at best.)

    An "exclusive write flag" situation is theoretically plausible, and your explanation might be plausible too. However from the analysis above, to me it looks more like the format of the EDL file is not what MP expects it to be. Nevertheless I could easily be wrong. Might be worth trying the standard Comskip launcher plugin to see if it makes any difference.
     
    mm1352000, July 9, 2016
  8. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,448
    Likes Received:
    4,663
    Ratings:
    +8,100 / 17
    Home Country:
    New Zealand New Zealand
    July 9, 2016 #7
    Ahhhh, it seems our documentation is misleading. :oops:

    According to our issue tracker, the feature has not yet been merged.
    https://issues.team-mediaportal.com/browse/MP1-4210

    The comments there give some idea of what happened.
    From a team/internal perspective, as far as I'm aware this has completely fallen off the radar...
     
    mm1352000, July 9, 2016
  9. erosco
    • Premium Supporter

    erosco MP Donator

    Joined:
    June 25, 2007
    Messages:
    122
    Likes Received:
    10
    Ratings:
    +11 / 0
    Home Country:
    July 9, 2016 #8
    Thanks - I did see that thread; however as it was in the release notes I figured it must have gone ahead.. I guess I should just give up :-(

    Shame, as it'd be a great feature to have.. I believe it's in kodi; although I love MP.
     
    erosco, July 9, 2016
  10. erosco
    • Premium Supporter

    erosco MP Donator

    Joined:
    June 25, 2007
    Messages:
    122
    Likes Received:
    10
    Ratings:
    +11 / 0
    Home Country:
    July 12, 2016 #9
    Just wanted to add a footnote that I worked out how to use git and cherry pick the required changes then build 1.15.0 with comskip support. Working fine - thanks

    I don't know if the team will reconsider this code sometime in the future but I hope so as I can't really see any side effects.

    Thanks again for your help mm1352000.

    Cheers
     
    erosco, July 12, 2016
  11. gonzo90017

    gonzo90017 Portal Pro

    Joined:
    September 3, 2012
    Messages:
    238
    Likes Received:
    13
    Ratings:
    +25 / 2
    Home Country:
    United States of America United States of America
    Show System Specs
    August 4, 2016 #10
    Might you consider sharing your build?
     
    gonzo90017, August 4, 2016
Page 1 of 2
Loading...
Similar Threads - Comskip meant live
  1. Bob C

    Comskip quit running after recording finishes.

    Bob C, May 15, 2018, in forum: Skins and Plugins Installer (MPEI)
    Replies:
    1
    Views:
    454
    Edalex
    May 15, 2018
  2. Ryan Birkenholz

    How to use comskip in MP2?

    Ryan Birkenholz, May 16, 2017, in forum: General
    Replies:
    6
    Views:
    1,191
    ge2301
    December 4, 2017
  3. aspik

    [fixed] ComSkip chapters + ZIP resource provider

    aspik, December 4, 2016, in forum: Archive
    Replies:
    2
    Views:
    71
    aspik
    December 12, 2016
  4. Gary_Francis

    Comskip Configuration: How To for V15

    Gary_Francis, June 29, 2016, in forum: MediaPortal 1 Plugins
    Replies:
    10
    Views:
    1,320
    Gary_Francis
    June 29, 2016
  5. Ere Maijala

    Show how far skipping moved when using ComSkip markers

    Ere Maijala, August 23, 2015, in forum: Feature Requests
    Replies:
    0
    Views:
    508
    Ere Maijala
    August 23, 2015

Users Viewing Thread (Users: 0, Guests: 0)

  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.
    Accept Learn More...
    Dismiss Notice

  • About The Project

    The vision of the MediaPortal project is to create a free open source media centre application, which supports all advanced media centre functions, and is accessible to all Windows users.

    In reaching this goal we are working every day to make sure our software is one of the best.

             

  • Quick Navigation

  • Support MediaPortal!

    The team works very hard to make sure the community is running the best HTPC-software. We give away MediaPortal for free but hosting and software is not for us.

    Care to support our work with a few bucks? We'd really appreciate it!