[solved] - MiniDisplayPlugin: Support for latest iMON VFD/LCD drivers | Page 10

Discussion in 'Submit: code patches (MediaPortal/TV-Server/etc.)' started by Stéphane Lenclud, January 7, 2014.

Thread Status:
Not open for further replies.
  1. Stéphane Lenclud
    • Team MediaPortal

    Stéphane Lenclud Development Group

    Joined:
    April 29, 2013
    Messages:
    2,563
    Likes Received:
    657
    Gender:
    Male
    Ratings:
    +1,270 / 8
    Home Country:
    Germany Germany
    Show System Specs
  2. Google AdSense Guest Advertisement



    to hide all adverts.
  3. azzuro
    • Team MediaPortal

    azzuro Test Group

    Joined:
    May 10, 2007
    Messages:
    8,512
    Likes Received:
    2,564
    Occupation:
    technician HVAC
    Location:
    France - IDF
    Ratings:
    +4,248 / 13
    Home Country:
    France France
    Show System Specs
    stéphane, in your Jira, don't forget to add all link inside it.
    because in few month, very hard to found the thread. who the bug or rework is needed.
     
    • Like Like x 2
  4. edterbak
    • Team MediaPortal

    edterbak Test Group

    Joined:
    March 4, 2008
    Messages:
    2,114
    Likes Received:
    736
    Gender:
    Male
    Occupation:
    Researcher
    Ratings:
    +1,178 / 5
    Home Country:
    Netherlands Netherlands
    Show System Specs
    • Thank You! Thank You! x 2
  5. edterbak
    • Team MediaPortal

    edterbak Test Group

    Joined:
    March 4, 2008
    Messages:
    2,114
    Likes Received:
    736
    Gender:
    Male
    Occupation:
    Researcher
    Ratings:
    +1,178 / 5
    Home Country:
    Netherlands Netherlands
    Show System Specs
    I noticed (also with the separate DLL version) when using Titan extended two the selected menu items dont show up correctly. It shows something like #menu... I don't know exactly. But its only with the titan main menu items using extended two. (my default) In any other plugin or selection it works fine.
    I suspect this is actually a skin issue?
     
    • Agree Agree x 1
  6. Stéphane Lenclud
    • Team MediaPortal

    Stéphane Lenclud Development Group

    Joined:
    April 29, 2013
    Messages:
    2,563
    Likes Received:
    657
    Gender:
    Male
    Ratings:
    +1,270 / 8
    Home Country:
    Germany Germany
    Show System Specs
    Definitely a skin/minidisplay settings issue. Not a problem of that iMON plugin. I had similar problems at some point but did not dig into it. See that post.
    Since I also noticed that MiniDisplay was working properly on the default skin after all. Is that an issue only with a certain variant of Titan?[DOUBLEPOST=1393669256][/DOUBLEPOST]Guys can we move this forward and merge to master?
     
  7. edterbak
    • Team MediaPortal

    edterbak Test Group

    Joined:
    March 4, 2008
    Messages:
    2,114
    Likes Received:
    736
    Gender:
    Male
    Occupation:
    Researcher
    Ratings:
    +1,178 / 5
    Home Country:
    Netherlands Netherlands
    Show System Specs
    yes, original titat, stock theme "extended two". ill inform skin dev. @ncoH :)

    This night I will describe the thing propperly in the titan forum for you.
     
  8. Sebastiii
    • Team MediaPortal

    Sebastiii Development Group

    Joined:
    November 12, 2007
    Messages:
    16,275
    Likes Received:
    6,223
    Gender:
    Male
    Location:
    France
    Ratings:
    +10,235 / 9
    Home Country:
    France France
    Show System Specs
    @HTPC_Sourcer :)

    Do you confirm using the new branch with old drivers is ok ?
    Thanks :)
     
    • Thank You! Thank You! x 1
  9. Stéphane Lenclud
    • Team MediaPortal

    Stéphane Lenclud Development Group

    Joined:
    April 29, 2013
    Messages:
    2,563
    Likes Received:
    657
    Gender:
    Male
    Ratings:
    +1,270 / 8
    Home Country:
    Germany Germany
    Show System Specs
    @Developers
    @Testers

    Guys it's time to get that stuff on master.
    The earlier it goes on master the more time we have to fix possible regressions before release.

    JIRA:
    https://issues.team-mediaportal.com/browse/MP1-4418

    The code review is there:
    https://github.com/MediaPortal/MediaPortal-1/pull/39/files
    https://issues.team-mediaportal.com/fisheye/cru/MP1CR-405

    If someone could just test that branch with a MiniDisplay driver that's not iMON that would be great.
    Either checkout that branch and compile it or patch a 1.7.1 with the binaries provided there:
    https://forum.team-mediaportal.com/...portal-1-1-3-1-2-1.95616/page-50#post-1080078
     
    • Thank You! Thank You! x 1
  10. Sebastiii
    • Team MediaPortal

    Sebastiii Development Group

    Joined:
    November 12, 2007
    Messages:
    16,275
    Likes Received:
    6,223
    Gender:
    Male
    Location:
    France
    Ratings:
    +10,235 / 9
    Home Country:
    France France
    Show System Specs
    Thanks for that :)

    But that not fully true :)
    We shouldn't use Master as testing branch but merge only stuff fully tested, so first it's to open an AREA51 thread (maybe already open) to have wider testing and then test regression and after that we can conclude that the branch is ready :)

    Of course, it can happen that even with wider testing something can be missed but in practice, the branch ready to be merged should be near to perfect :) (read no regression).

    Don't get me wrong here :) i'm just explain that we need to have good testing :)
     
    • Agree Agree x 1
  11. Stéphane Lenclud
    • Team MediaPortal

    Stéphane Lenclud Development Group

    Joined:
    April 29, 2013
    Messages:
    2,563
    Likes Received:
    657
    Gender:
    Male
    Ratings:
    +1,270 / 8
    Home Country:
    Germany Germany
    Show System Specs
    It has received a lot of testing from at least 5 MiniDisplay users plus myself. All that's left is code review and test another driver/display to check for regressions. I really don't see the point of opening yet another thread about that. All dev and testers should have already been informed and most interested users too. All that's left really is for someone with the authority to push a button and get it merged.
    Hope we can get this sorted before Sunday.
     
    • Thank You! Thank You! x 2
    • Like Like x 1
Loading...
Thread Status:
Not open for further replies.

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.
    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.

             

  • 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!