[Pending] - Added process priority setting to Comskip Launcher plugin.

Discussion in 'Submit: code patches (MediaPortal/TV-Server/etc.)' started by cwchapma, May 11, 2013.

Page 1 of 4
  1. cwchapma

    cwchapma Portal Pro

    Joined:
    October 15, 2006
    Messages:
    56
    Likes Received:
    17
    Ratings:
    +22 / 0
    Home Country:
    Canada Canada
    Show System Specs
    May 11, 2013 #1
    I've submitted a pull request for a change to the comskip launcher plugin to allow/default to running comskip with a lower than normal thread priority. It made a huge difference to my mediaportal setup and I would love to share :)

    https://github.com/MediaPortal/MediaPortal-1/pull/23



    Comments appreciated,
    Clint
     
    cwchapma, May 11, 2013
    • Like Like x 6
    • Thank You! Thank You! x 1

  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. cwchapma

    cwchapma Portal Pro

    Joined:
    October 15, 2006
    Messages:
    56
    Likes Received:
    17
    Ratings:
    +22 / 0
    Home Country:
    Canada Canada
    Show System Specs
    October 29, 2013 #2
    cwchapma, October 29, 2013
  4. regeszter
    • Premium Supporter

    regeszter Retired Team Member

    Joined:
    October 29, 2005
    Messages:
    5,335
    Likes Received:
    2,863
    Ratings:
    +5,152 / 18
    Home Country:
    Hungary Hungary
    October 29, 2013 #3
    Hi,

    the TVE3 is in code freeze state so we can't do any changes until TVE3.5 is released.
     
    regeszter, October 29, 2013
  5. cwchapma

    cwchapma Portal Pro

    Joined:
    October 15, 2006
    Messages:
    56
    Likes Received:
    17
    Ratings:
    +22 / 0
    Home Country:
    Canada Canada
    Show System Specs
    October 30, 2013 #4
    I've tried to find info on TVE 3.5 and haven't had any luck. Is there info somewhere on what the features of 3.5 are or when it is expected to be released?

    Thanks for letting me know.

    Clint
     
    cwchapma, October 30, 2013
  6. Sebastiii
    • Team MediaPortal

    Sebastiii Development Group

    Joined:
    November 12, 2007
    Messages:
    16,312
    Likes Received:
    6,256
    Gender:
    Male
    Location:
    France
    Ratings:
    +10,290 / 9
    Home Country:
    France France
    Show System Specs
    October 30, 2013 #5
    If i'm not wrong the code will be easy to port to TVE35 so maybe we should include it for TVE3 (if all seems in good way TVE3 will be the last with 1.6 Release, i mean code freeze and we can focus on TVE35).

    @mm1352000 @elliottmc ? :)

    Ps : Like we have a big work to adapt current TVE35 against actual TVE3, i think that patch will be a very little change :)
     
    Sebastiii, October 30, 2013
  7. elliottmc
    • Premium Supporter

    elliottmc Retired Team Member

    Joined:
    August 7, 2005
    Messages:
    14,927
    Likes Received:
    4,011
    Gender:
    Male
    Location:
    Cardiff, UK
    Ratings:
    +5,503 / 13
    Home Country:
    United Kingdom United Kingdom
    Show System Specs
    October 30, 2013 #6
    But every additional change adds to the work. This really is down to mm1352000.
     
    elliottmc, October 30, 2013
    • Like Like x 1
  8. Holzi
    • Team MediaPortal

    Holzi Super Moderator

    Joined:
    April 21, 2010
    Messages:
    7,930
    Likes Received:
    1,591
    Gender:
    Male
    Location:
    Ba-Wü
    Ratings:
    +2,228 / 8
    Home Country:
    Germany Germany
    Show System Specs
    October 30, 2013 #7
    Holzi, October 30, 2013
  9. mm1352000
    • Team MediaPortal

    mm1352000 Development Group

    Joined:
    September 1, 2008
    Messages:
    21,541
    Likes Received:
    4,752
    Ratings:
    +8,220 / 17
    Home Country:
    New Zealand New Zealand
    October 30, 2013 #8
    The tricky stuff will be the stuff that involves TvBusinessLayer and settings, because TvBusinessLayer no longer exists.

    I don't want to be the bad guy but we need to draw the line somewhere. Personally I don't use comskip and I don't know that area of the code. As with MySQL, I'm not going to stop you putting it into TVE 3 as long as you are willing to merge it to TVE 3.5 (assuming the patch is accepted).
     
    mm1352000, October 30, 2013
    • Like Like x 2
    • Thank You! Thank You! x 2
    • Agree Agree x 2
  10. markus_g

    markus_g Portal Pro

    Joined:
    August 12, 2006
    Messages:
    104
    Likes Received:
    38
    Gender:
    Male
    Ratings:
    +56 / 0
    Home Country:
    Australia Australia
    Show System Specs
    October 30, 2013 #9
    Hi, I have just started using Comskip and have found this is really needed. Tsreader on my setup doesn't seem to cope very well when Comskip is running.
    Regards,

    Markus.
     
    markus_g, October 30, 2013
  11. mbuzina
    • Premium Supporter

    mbuzina Retired Team Member

    Joined:
    April 11, 2005
    Messages:
    2,839
    Likes Received:
    627
    Location:
    Germany
    Ratings:
    +723 / 4
    Home Country:
    Germany Germany
    Show System Specs
    October 30, 2013 #10
    Are TV Server Plugins also frozen? I thought we have a plugin structure to avoid such issues...

    Unless TVE 3.5 contains a different ComSkip Launcher it should not be a big issue, shouldn't it?
     
    mbuzina, October 30, 2013
Page 1 of 4
Loading...
Similar Threads - Added process priority
  1. BrownGhost

    MP-TVSeries, Failed to save xml to cache, and not adding any info to tvseries from tvdb

    BrownGhost, November 28, 2019, in forum: My TVSeries
    Replies:
    4
    Views:
    348
    BrownGhost
    November 28, 2019
  2. Gcarr

    How best to get cover art for tracks to show up on the Audio Menu (Latest Added, Unplayed)?

    Gcarr, May 10, 2019, in forum: General
    Replies:
    9
    Views:
    285
    henso
    May 12, 2019
  3. MediaPortal-Bot

    0003724: Graphbuilding TV Side (Post Process Fix / Close Caption / adding correct filter in...

    MediaPortal-Bot, May 11, 2012, in forum: Bugtracker Feed
    Replies:
    0
    Views:
    531
    MediaPortal-Bot
    May 11, 2012
  4. MediaPortal-Bot

    0003724: Graphbuilding TV Side (Post Process Fix / Close Caption / adding correct fil

    MediaPortal-Bot, September 23, 2011, in forum: Bugtracker Feed
    Replies:
    0
    Views:
    503
    MediaPortal-Bot
    September 23, 2011
  5. gperkinson

    Adding Explorer To TV Server Processe Tab To Stop Standby

    gperkinson, November 18, 2008, in forum: Television (MyTV frontend and TV-Server)
    Replies:
    24
    Views:
    2,441
    Owlsroost
    November 4, 2009

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!