Is it possible to get more info from Ice EPG on shows?

[adoxa]

Updated the previous attachment. Hopefully I only missed the one.

[peteru]

forum doesn't support .ipk

It should now.

[paullings]

Hi Adoxa,

I have installed the plugin version however the extra info is not displaying in either OverlayHD or easy-aus skins.
Patch version works fine though.

[Grumpy_Geoff]

Updated the previous attachment. Hopefully I only missed the one.

Nup

Code: Select all

exceptions.NameError: global name 'parental_ratings' is not defined

{597}< 16281.150> [IceTV] 2019-07-23 10:35:37: Error trying to fetch: global name 'parental_ratings' is not defined

[adoxa]

Updated the previous attachment (just .ipk now, thanks peteru!). Third time's the charm...

[Grumpy_Geoff]

Third time's the charm...

Your goolies are safe

[stephenh]

Updated the previous attachment (just .ipk now, thanks peteru!). Third time's the charm...

1.0.2 still doesn't seem to be working for me on my U4 ... I don't get the [Rpt] flag showing or the extended "Drama, Music, Romance, Movie" tags, I just get "Drama". I removed the Plugin and re-installed and then compared the "info" page, seems to stay the same ...

Is there anything else I need to turn on or adjust?

StephenH

[Grumpy_Geoff]

Is there anything else I need to turn on or adjust?

Restart the GUI (or reboot) as the box will then fetch the EPG.
Alternatively, disable IceTV, bring up the EPG and observe it's empty, then re-enable IceTV and 'Fetch EPG and update tImers now'.

[stephenh]

Restart the GUI (or reboot) as the box will then fetch the EPG.
Alternatively, disable IceTV, bring up the EPG and observe it's empty, then re-enable IceTV and 'Fetch EPG and update tImers now'.

Yep, done all that, and IceTV is definitely working ... I can see it has different info to the DVB EIT info when I turn it off ... but I'm still not sure I'm getting extended info as my screen looks different to yours.
(Scratches head)

[Paul_oz53]

Restart the GUI (or reboot) as the box will then fetch the EPG.
Alternatively, disable IceTV, bring up the EPG and observe it's empty, then re-enable IceTV and 'Fetch EPG and update tImers now'.

Yep, done all that, and IceTV is definitely working ... I can see it has different info to the DVB EIT info when I turn it off ... but I'm still not sure I'm getting extended info as my screen looks different to yours.
(Scratches head)

Installed it, restarted the GUI and viola! As per picture above.

What I find is easily noticeable is for a movie, an extra line beginning "Cast:" appears at the end of the synopsis with a list of names. Also, a year in brackets appears in the title line. These are the quick "tells" in any version of the EPG. Easiest seen in the single EPG.

[prl]

Yep, done all that, and IceTV is definitely working ... I can see it has different info to the DVB EIT info when I turn it off ... but I'm still not sure I'm getting extended info as my screen looks different to yours.
(Scratches head)

Do you mean your screen for The Piano?

The IceTV data has credits on only about 1/3 of the entries (in an about year-old data capture). That's shows with any credits, cast or director.

[Grumpy_Geoff]

...restarted the GUI and viola!

Now you can play a musical instrument after simply restarting the UI? What achievement can you reach after rebooting?

[prl]

...restarted the GUI and viola!

Now you can play a musical instrument after simply restarting the UI? What achievement can you reach after rebooting?

Clog dancing

[Grumpy_Geoff]

Clog dancing

re-booting - I got it

[Paul_oz53]

Clog dancing

re-booting - I got it

The penny finally dropped here too

[prl]

[stephenh]

Do you mean your screen for The Piano?

The IceTV data has credits on only about 1/3 of the entries (in an about year-old data capture). That's shows with any credits, cast or director.

Yes, I was directly comparing the screen shot from @Grumpy_Geoff for The Piano to my screen for the same show yesterday “as it happened” live.
I was seeing the same info exactly for most fields, except I was missing the [Rpt] flag, and also down the bottom instead of seeing “Drama Music Romance Movie” I was just seeing “Drama”.
Hence my confusion ...

[Grumpy_Geoff]

I was seeing the same info exactly for most fields, except I was missing the [Rpt] flag,

Have you set the custom repeat flag on the IceTV website under My Account / My Recorders, in the 'Custom repeat flag' setting for your device?
I have it set to be at the beginning of the short description (you can also have it in the title or extended description).

and also down the bottom instead of seeing “Drama Music Romance Movie” I was just seeing “Drama”.
Hence my confusion ...

I don't know if it makes a difference, but have you set the 'Country for EPG event genre information' in the MENU>>Setup>>TV>>EPG settings?

[prl]

and also down the bottom instead of seeing “Drama Music Romance Movie” I was just seeing “Drama”.
Hence my confusion ...

I don't know if it makes a difference, but have you set the 'Country for EPG event genre information' in the MENU>>Setup>>TV>>EPG settings?

If the wrong country is selected you'd most likely get no genre information, rather that a single genre item of the available list.

The pertinent question is more likely to be: "Are you using the OverlayHD skin?"

OverlayHD only displays the first item in the IceTV list of genres. This is because the mechanism to display the whole genre list does not exist on some of the other enigma2 variants that OverlayHD is used on.

[Grumpy_Geoff]

The pertinent question is more likely to be: "Are you using the OverlayHD skin?"

OverlayHD only displays the first item in the IceTV list of genres. This is because the mechanism to display the whole genre list does not exist on some of the other enigma2 variants that OverlayHD is used on.

Ah okay.
The answer to your pertinent question is likely to be "yes", based on this post.

[stephenh]

Outstanding detective work by all of you!

Have you set the custom repeat flag on the IceTV website under My Account / My Recorders, in the 'Custom repeat flag' setting for your device?
I have it set to be at the beginning of the short description (you can also have it in the title or extended description).

Indeed that made a difference, I set it the same as you and it now appears on the U4.

If the wrong country is selected you'd most likely get no genre information, rather that a single genre item of the available list.

The pertinent question is more likely to be: "Are you using the OverlayHD skin?"

OverlayHD only displays the first item in the IceTV list of genres. This is because the mechanism to display the whole genre list does not exist on some of the other enigma2 variants that OverlayHD is used on.

Yes, setting the Skin to something other than OverlayHD did indeed bring back the multiple genres.

I just have to decide what I like the most, as in "least worst" compromise.
I like OverlayHD by far the most out of the Skins, but having the multiple Genres shown is useful as well, so no easy answer there ...

Thanks for the help as always, "mysteries" now solved.

StephenH

[prl]

I'd like Full-Metal-Wizard a whole lot more if it had something better-looking than its "boiled sweet" coloured buttons.

The things I don't like about OverlayHD are that its main menu (liveTV MENU) and its main settings menu are a bit crowded, and its handling of background colour in popups isn't all that great. But it is cleaner and more consistent generally than the pre-loaded skins, has a lot of flexibility in other colour choices and generally shows the benefits of a complete re-write.

[stephenh]

But it is cleaner and more consistent generally than the pre-loaded skins, has a lot of flexibility in other colour choices and generally shows the benefits of a complete re-write.

Agreed!

[peteru]

I'd like Full-Metal-Wizard a whole lot more if it had something better-looking than its "boiled sweet" coloured buttons.

Yes, those buttons are less than ideal. The colour choice is also a bit intense and does not really mesh too well with the rest of the skin.

If anyone wants to come up with alternative design for those buttons, I'd be happy to merge the changes if there is community support.

[prl]

If anyone wants to come up with alternative design for those buttons, I'd be happy to merge the changes if there is community support.

I agree. Unfortunately, I don't have the graphic design skills for even that

[prl]

Adoxa, in case you've missed it, I think that recent changes (still only in beta firmware) to the genre_remaps dict in Plugins/SystemPlugins/IceTV/plugin.py mean that with IceTVDesc installed, errors are thrown for any genres that are in the (now) genre_remaps["AUS"] dict.

See Grumpy_Geoff's description of the error, and our discussion about it in the IceTV forum.

[adoxa]

Updated to match IceTV.

[prl]

Thanks

[Grumpy_Geoff]

Updated to match IceTV.

Thanks.
Cooking with gas again. Only the "Politics" lookup failure was reported.

[prl]

A new version of the IceTV plugin, 20201026, with support for the new genres has been merged into the source repository and should be in the next firmware update.

In the meantime, IceTV have started using a second instance of the new genres, "History", so debug logs in the old firmware may now flag "Politics" and "History" as unknown.

Adoxa: this change affects the contents of the genre tables, but not their structure, and should not affect IceTVDesc.

[Grumpy_Geoff]

Adoxa,

With IceTVDesc installed, the IceTV plugin ver 20201031 is being very grumpy...

Code: Select all

...    
  File "/usr/lib/enigma2/python/Plugins/SystemPlugins/IceTV/plugin.py", line 604, in doWork
  File "/usr/lib/enigma2/python/Plugins/SystemPlugins/IceTV/plugin.py", line 802, in processShowsBatched
  File "/usr/lib/enigma2/python/Plugins/SystemPlugins/IceTV/plugin.py", line 718, in convertChanShows
exceptions.AttributeError: 'tuple' object has no attribute 'get'

[adoxa]

This should make it geoff...

[Grumpy_Geoff]

This should make it geoff...

Yep, it does. Thanks.

[Paul_oz53]

This should make it geoff...

Yep, it does. Thanks.

+1!

I was getting random crashes on 3 machines until I realized I needed to update IceTV descriptions. The logs were full of errors and restarts triggered by IceTV until I updated. Logs looking pretty clean now.

Thanks Adoxa.

[adoxa]

Here's a version that should be more adaptable, if slightly slower (it shouldn't be significant, but I'm not sure). This should make it work with the next update (which has sanity checks) and all future updates (until there's another major restructure). (It's still not for anything earlier than late October, though.)

[prl]

Maybe the IceTV plugin could have getTitle(), getSubtitle() & getDescription() methods that would make what you're trying to do easier and cleaner, and less susceptible to other changes in the plugin code. I'm assuming that a signature of getMetadataItem(self, show) would work for you.

It's still on my todo list to encode as much as possible of the IceTV metadata into the extended description as extended description items tagged by an item_description, along with access methods to extract them and leave access to the "normal" extended description unchanged.

That would allow the main UI code to add extra metadata in the appropriate places. It might even allow the display of the movie poster images in the IceTV database.

My changes to make IceTV fetch batch sizing dynamic on the amount of memory available should not affect IceTVDesc. Now that the sanity checks have been merged, I'll try to submit the dynamic batch sizing today.

[adoxa]

That might simplify things, but I don't think there's any need. If you really want to make it easier for me, (add an option and) do it all in IceTV in the first place. Here's what the new version does (the previous versions simply duplicated and replaced the function outright).

Code: Select all

def convertChanShows(self, oldfunction, shows, mapping_errors):
	for show in shows:
		extended = show.get("desc", "").rstrip().replace("\r", "") + "\n"
		year = show.get("date", "0")
		if year != "0":
			title = show.get("title", "")
			if title.startswith("Movie: "):
				title += " (%s)" % year
				show["title"] = title
			else:
				extended += "\nYear: " + year
		try:
			cast = ", ".join([actor["name"] for actor in show["credits"]["actors"]])
			if cast:
				extended += "\nCast: " + cast
		except:
			pass
		show["desc"] = extended
	return oldfunction(self, shows, mapping_errors)

Simple enough, assuming iterating through shows is negligible (or if it's done in the background it probably doesn't even matter) and convertChanShows doesn't change (this is what broke 1.0.3, since it used to be done in makeChanShowMap).

[prl]

But that would then add it for everyone, whether they want it or not.

[prl]

Adoxa, the pull request for the IceTV dynamic batching is here: https://bitbucket.org/beyonwiz/easy-ui- ... nd-on/diff