- Thread starter
- #261
Re: HTPCInfo plugin for MediaPortal v0.9.0
Hi tourettes,
thanks for your professional feedback, some statements to your listed issues:
For all realtime tasks (performance counter, temperature alerts, disk checking etc.) I use the BackgroundWorker() function from .NET. Only static tasks, which are needed only at startup, are running in Process() method.
That's a good idea. It is possible to implement some triggers to identify, if TV or video is playing. In my opinion, I must give the end user a option to switch on/off this feature, because some users want display HTPCInfo informations inside OSD.
Inside the configuration dialog, the end user has the possibility to set refresh periods and priorities. But ok, it is not a problem to set a lower priority by default.
Your suggestions and ideas will be part of the next release, more feedback are very welcome.
Cheers, Micropolis ;-)
Hi tourettes,
thanks for your professional feedback, some statements to your listed issues:
1) Process() method overloading should be done as less as possible (only handle messaging between MP and the plugin there). On every single video frame rendering the Process() is run thru and if there is some code paths that cause few ms delays it could already cause dropped frame(s). We have only 20ms per frame time for the video decoding, running process() code, trying to match the v-sync, rendering the video frame and on top of that OSD etc. So, use as less code that runs in MPMain thread (easy to see if you put logging into every method, MP will log the used thread)
For all realtime tasks (performance counter, temperature alerts, disk checking etc.) I use the BackgroundWorker() function from .NET. Only static tasks, which are needed only at startup, are running in Process() method.
2) All non-critical background processing should be halted when MP is palying a video (g_player should give this info)
That's a good idea. It is possible to implement some triggers to identify, if TV or video is playing. In my opinion, I must give the end user a option to switch on/off this feature, because some users want display HTPCInfo informations inside OSD.
3) All background threads should be running lower priority that the MP itself, this will give smoother GUI animations etc.
Inside the configuration dialog, the end user has the possibility to set refresh periods and priorities. But ok, it is not a problem to set a lower priority by default.
Your suggestions and ideas will be part of the next release, more feedback are very welcome.
Cheers, Micropolis ;-)