[Approved] Store EPG only for Channels on same Transponder (1 Viewer)

[snowball]

Hi,

In germany the ARD group sends the next 2 Programs of the ZDF Group also on their transponders. this results that you have only 2 programs in the EPG for ZDF channels. similar thing happens on SKY germany. they seem to send EPG for all their channels on all transponders but only detail (description) on the transponders where the channel is located. As timeshift EPG is not an option for me i have patched the TV service in a way that it only updates the DB for channels received from EPG that are on the same transponder.
That seems to be quite a german problem, but would would be good when you could have a look if it can be integrated into the main program.
not sure if i have done all correctly ... but its the first time that i post a patch.

At least it works for me.

currently i have patched it against 1.1.1 as i have only one "productive TVServer" running this version at the moment.

Chears Snow

**************************************************************************
update:
attached a version for the current 1.2.0 SVN with the recommended code changes

 

[gemx]

Seems like a nice addition.

Could you please update the patch so that it includes the following:

1) setting in SetupTV to enable/disable the setting
2) Move the logic for this switch into the dbupdater class



for your work so far

 

[snowball]

will see what i can do. i am not that familiar with all the MP code. will take a while to work into all the logic.
I had a look at moving the change to the dbupdaterclass.
from what i can see this class is not aware of the current tunning details. so moving this to the dbupdater class would need to hand over the tunning details. I think this will have more impact on the code. i have to look if the timeshift epg grabbing can be changed too.

Cheers Snow

 

[snowball]

Hi gemx,

added the config switch in the configuration for tvservice.
i did not moved the logic for the filtering to the epgupdater class. i think that this might lead to problems. as there are events fired although we filter out the EPG program in the epgupdater class. this might lead to inconsistencies as plugins might react on the events fired but will not find an entry in the DB. i moved the filtering to the implementations of the different DVB versions. this basically does the filtering like an input filter (all Plugins/DB will have the same information later)
at the moment i could test it only for DVBS. but there should not be a big difference for the other DVB types.
here the patch for the 1.1.1 version View attachment MP1.1.1_epg_transponder_v0.1.patch
i also tried to patch against the 1.2.0 SVN but had some errors so i also created another patch file for the svn version View attachment MP1.2.0_svn_epg_transponder_v0.1.patch

again pls be patient with me in case i did not followed some rules. let me know what to change.
Would be nice when you can have a look if this can be incorporated.

ps.:
completely missed in my first post.
THANKS for all the the good work MP Team!

Cheers Snow

 

[riksmith]

Question: What is the default if you do not set the config yourself? I would like the functionality to stay as i currently is unless you choose the option yourself.

 

[snowball]

I currently set it to disabled by default. so classic methode all will be grapped.

Cheers Snow

 

[disaster123]

AW: Store EPG only for Channels on same Transponder

snowball
Really nice patch. But if you look at: https://forum.team-mediaportal.com/...ponder-mp1.2.0_svn_epg_transponder_v0.1.patch you'll see that in TVLibrary/SetupTv/Sections/Epg.Designer.cs most parts of the code are just whitespace moves. Could you correct this and attach a proper patch?

 

[SilentException]

Yes, please make the patch following MePo coding standards

 

[romuz]

Proposal: With this addition for easier maintain we should add a transponder identifier to channel assignment to grab epg layout.
Its enough to grab epg from single channel per transponder but this information is hiden from user.

 

[snowball]

All,

thx for the advices. i've changed the patch accordingly and attached in the first post.
Btw... when i have done the changes, do i have to change the status of the thread to evaluate again?

romuz
i did not completely got what you want to achieve.
can you explain a bit detailed?

Cheers Snow