[solved] MP1-4585 Generic HID input device support for MCE buttons (1 Viewer)

Stéphane Lenclud

Development Group
  • Team MediaPortal
  • April 29, 2013
    2,565
    1,290
    Germany Germany
    Country flag
    Thanks guys for your feedback. I did not get a chance to work on those changes nor answer some of your points here because I've been busy doing a major hardware upgrade to my HTPC. The beast is now back in the living room and I've resumed testing of those changes.
    I've taken out the iMON IR module and replaced it with a PCS-MCE.
    First thing I noticed is that it did not work with Generic HID for some reason, though it was obviously working on my development machine.
    It turns out that Shortcut'er plugin was causing the problem. I'm not sure what exactly that plug-in does but it seems it intercepts the WM_INPUT and thus prevents the Generic HID implementation to function.
    However I don't need Shortcut'er with a 'proper' remote device, though it was useful when using iMON.
     

    Stéphane Lenclud

    Development Group
  • Team MediaPortal
  • April 29, 2013
    2,565
    1,290
    Germany Germany
    Country flag
    Sorry, I think you may have missed my point. I'll try again... :)
    My point is that somebody who is using MCE + old generic HID now will have MCE + new HID enabled after upgrade. It isn't that people would necessarily apply that configuration intentionally. Rather, it will be the natural consequence of a standard upgrade for people who currently use that [potentially common] configuration. That's why I asked what would happen for them. Suddenly having many/all button presses repeated would not be good.
    I see, so you concerns are that I have recycled the settings entries from the former 'General HID'. I thought about that and decided to be lazy about it and not bother changing them. Either way users will have to go back in the Configuration and fix their Remote settings. I'm going to rename the settings entries to make it cleaner I guess.

    If the legacy implementations are not required then I'm strongly in favour of removing them as fast as possible. Less configuration => less setup time, less confusion => better. :)
    I totally agree with that. If all goes well we could withdraw the 'Microsoft MCE' implementation in 1.12 or 1.13.

    I only mentioned it because you seem to have added that function definition. ;)
    Ooops, then I try cleaning up my own mess :)
     

    The_Stig

    Retired Team Member
  • Premium Supporter
  • April 5, 2005
    2,176
    428
    Country flag
    Hey Stephane,

    as you're on this way...I am personally using a standard MCE remote and am quite happy with it. As far as I understand, this change won't bring up any new features for me, or am I wrong?

    But: what I always found no really nice regaring the remote possibilities of MP is that you can't map Macros, Batches or Keyboard-combinations to a button on the remote within MP configuration. Do you see any way to include such things in your rework?
     

    Stéphane Lenclud

    Development Group
  • Team MediaPortal
  • April 29, 2013
    2,565
    1,290
    Germany Germany
    Country flag
    as you're on this way...I am personally using a standard MCE remote and am quite happy with it. As far as I understand, this change won't bring up any new features for me, or am I wrong?
    You are right it should not change much for you.
    Though it should make it somewhat easier for you to configure your remote if you fancy editing XML files.

    But: what I always found no really nice regaring the remote possibilities of MP is that you can't map Macros, Batches or Keyboard-combinations to a button on the remote within MP configuration. Do you see any way to include such things in your rework?
    It's outside the scope of those changes.
     

    Stéphane Lenclud

    Development Group
  • Team MediaPortal
  • April 29, 2013
    2,565
    1,290
    Germany Germany
    Country flag
    P.S.: while writing this post I happened to notice that the entry point for GetRawInputData() in Win32API.cs is incorrect. Looks like that function isn't used (there's another definition that is used instead), but might pay to fix it (or remove it).
    Well spotted, I fixed it. The legacy bits of raw input in Win32API.cs could be removed whenever we get rid of 'Microsoft MCE'.

    I just published a new build on the first post that's adding a bit of logs and fixed the settings entries to address @mm1352000 concerns.
    If you get a chance please smoke test that last build. I'll be wanting to merge those changes as they are now.

    @high it seems JIRA is running quite slow and fisheye is down (good riddance :oops: you know I love that tool )

    Diffs can be found there:
    https://github.com/MediaPortal/MediaPortal-1/commit/edfc02ea543afea35c0a1565419f74a1c531929c
     

    Stéphane Lenclud

    Development Group
  • Team MediaPortal
  • April 29, 2013
    2,565
    1,290
    Germany Germany
    Country flag
    But: what I always found no really nice regaring the remote possibilities of MP is that you can't map Macros, Batches or Keyboard-combinations to a button on the remote within MP configuration. Do you see any way to include such things in your rework?
    I believe you can have any number of actions associated with a button. Would that not do for your needs?
    Though I'm not sure how that works in practice, I'm not sure how and if the order of executions of actions is guaranteed for instance.

    @The_Stig If you want to discuss that further please open another thread and feel free to tag me on there if you think I can bring something to that topic.
     

    The_Stig

    Retired Team Member
  • Premium Supporter
  • April 5, 2005
    2,176
    428
    Country flag
    @The_Stig If you want to discuss that further please open another thread and feel free to tag me on there if you think I can bring something to that topic.
    Thanks. But I guess it would be a massive overwork what I am thinking about. No need to bother the good old lady MP1 with this change I guess.
     

    Users Who Are Viewing This Thread (Users: 0, Guests: 1)

    OP Similar threads Forum Replies Date
    ajs [News] MediaPortal 1.26 Pre-Release ready for testing! News and Announcements 0
    C Hauppauge HVR-1265 -- does it work with MP1? MediaPortal 1 Talk 1
    ajs MP2Web - MP1 compatibility MediaPortal 1 20
    T MadVR + LAV Filters, Perfect in MPC-BE, Bad in MP1 MediaPortal 1 Talk 13
    P MP2: US user, OTA - can NOT get SchedulesDirect plugin to work on MP1 or MP2 --> how do I get OTA schedule? General 5
    framug [solved] Some changes/fix in MP1 configuration client and server Submit: code patches (MediaPortal/TV-Server/etc.) 14
    S TV-Server MP1 - Automatic search doesn't work Installation, configuration support 2
    C Streaming/transcoding MP2 vs MP1? General 4
    ajs [fixed] 5015: Improving the MP1 installer, providing nicer GUI :) MediaPortal 1 138
    C Streaming to clients over Wifi. MP1 or MP2? Newcomers Forum 0
    S Shortcuts MP1 My Recorded & TV Guide General 10
    azzuro [WiP] MP1-4999 - Big changes with many related change at once by Sebastiii MediaPortal 1 22
    ajs [WiP] MP1 X64 Version MediaPortal 1 70
    joecrow How about a Launcher plugin for MP1? MediaPortal 1 6
    wizard123 [solved] MP1 Build help MediaPortal 1 21
    ajs [News] MediaPortal 1 - 1.24 Released! News and Announcements 3
    CyberSimian [fixed] MP1-4990 : Improve horizontal scrolling in the EPG MediaPortal 1 10
    azzuro [WiP] MP1-5019: Replace MySQL with MariaDB MediaPortal 1 40
    catalpa Umsteiger von MP1: Schwierigkeiten Allgemeines Support- und Diskussionsforum 17
    CyberSimian [solved] MP1-4976 Built-in skin variables for current date MediaPortal 1 8
    rbo [solved] Umsteiger MP1 zu MP2 Allgemeines Support- und Diskussionsforum 67
    azzuro [Info] MP2 TVE inside MP1 repo MediaPortal 2 12
    G MP1 TV Server "failed to upgrade the database" using MS Sql Server Installation, configuration support 3
    lisag Can I get MP1 to open in the Recorded TV screen? MediaPortal 1 Talk 2
    M MP1 stops responding after Starting Plugins General Support 14
    Similar threads

























    Top Bottom