add-ons / service.iptv.manager

Integrate TV and radio channels with EPG data from other add-ons in Kodi PVR.
https://github.com/add-ons/service.iptv.manager/wiki
GNU General Public License v3.0
95 stars 28 forks source link

Show TV/Radio group(s) in the channel listing description in PVR #76

Open dagwieers opened 3 years ago

dagwieers commented 3 years ago

Since we now have many add-ons adding channels to PVR IPTV Simple (and in some cases the same channel) it would be useful to users to know which group(s) the selected channel is part of.

screenshot013

cc @phunkyfish

phunkyfish commented 3 years ago

I don’t see the issue. They just need to select the group they are interested in instead of all channels.

dagwieers commented 3 years ago

It is not so much an issue, than it is an inconvenience. Selecting a group just to find if a channel is part of that group is counter intuitive and may not actually help that much. We have add-ons that expose 50 or more channels (TV Vlaanderen, Yelo).

Also, not every household member knows about channel groups, and this would require you to select all groups again after you stopped watching the channel you were looking for.

Being able to see in Kodi which group the selected channel belongs to is a lot more convenient.

phunkyfish commented 3 years ago

A channel can belong to multiple groups or a single group. I’m not sure how to depict what you want in that view.

michaelarnauts commented 3 years ago

I think we should solve this issue another way. For a user, it should not matter what group a channel is in. Duplicate channels should be hidden or merged somehow. Maybe something more in the lines of #43

dagwieers commented 3 years ago

@michaelarnauts The purpose is not just to differentiate between channels, but to know what group they are in. (For IPTV Manager it would identify the add-on it relates to)

43 is a different issue.

dagwieers commented 3 years ago

A channel can belong to multiple groups or a single group. I’m not sure how to depict what you want in that view.

The easiest option is to have the group names listed in the right pane.

| Groups: Catch-up TV & More, Belgium

michaelarnauts commented 3 years ago

A channel can belong to multiple groups or a single group. I’m not sure how to depict what you want in that view.

The easiest option is to have the group names listed in the right pane.

| Groups: Catch-up TV & More, Belgium

This would be an issue for Kodi then. I don't think there is a way to pass a description with a Channel.

dagwieers commented 3 years ago

It is ~an issue~a feature request for Kodi. This is a tracking ticket.

phunkyfish commented 3 years ago

Why is this an issue for kodi? A channel can belong to any number of groups. It’s not feasible to show a list with possibly many items in the right pane. Nor is it correct to do so.

The UI is structured so the group is at the top, channel list on the left and channel details on the right.

The group are you in is all channels.

dagwieers commented 3 years ago

Why is this an issue for kodi?

It is not an issue, it is a feature request.

When we discussed this earlier, one option was to have it added to the EPG description. For this we would not require a change to Kodi,. But decided that having it in Kodi would be visually more attractive (and skins could opt to use it differently). That is why we think this is better served in Kodi.

A channel can belong to any number of groups. It’s not feasible to show a list with possibly many items in the right pane. Nor is it correct to do so.

Most (if not all) channels are not part of many groups. But in the case there are many, we can do exactly the same thing as the Kodi category (they are cut off when they go beyond the foreseen box). At least in the majority of cases it would help understand which group(s) a channel belongs to. I know it would be extremely useful to our Kodi users.

The UI is structured so the group is at the top, channel list on the left and channel details on the right.

Perfect, this is a channel detail, so it belongs on the right pane as I suggested.

phunkyfish commented 3 years ago

You are thinking of the UI in the wrong way. The parent of the entire screen is the channel group. What you are asking for is to display the other channel groups this channel belong too but only for all channels?

The skin doesn’t know this is the all channels group. It just display the channels listed without any knowledge of any other groups.

In short, what you are asking for is not an easy problem to solve in a sensible way.

dagwieers commented 3 years ago

I appreciate your input (that is why I added you in cc), but we are just discussing an idea here. Which starts with what is desirable.

I understand that the current code does not offer this information readily available. And that Skins do not have access to this information already. (Honestly, if they did that would be fantastic and changes the course of this process).

But what you are doing here is dismissing an idea on the basis that it would require changes to Kodi and skins, not on its own merits. That is the world upside down IMO. The code assists in what we think is needed for the UI, not the other way around.

Making the list of groups available as a variable to the skin, per channel/id, is in itself not a hard problem to solve. Putting this information on the right pane in a skin neither. But that is not even the purpose of this tracking ticket. Its purpose is to have an entry point for discussing a perceived inconvenience (something we can point people to), discussing possible options openly and eventually improve Kodi in this direction.

We have worked like this before for all our add-on projects, and it has lead over time to various improvements in Kodi, ISA, ISH and Up Next.

phunkyfish commented 3 years ago

Fair point. This has pivoted how I think about that channels UI now. Because now many thousands of channels are now a distinct reality whereas before it was something that you would try in earnest to avoid.

I have a feeling that what is needed is a different way to represent the data.

dagwieers commented 3 years ago

@phunkyfish I am sure how we think about this will change when this gets used more widely. I would also like to see what @matthuisman's view is on this.

phunkyfish commented 3 years ago

As a workaround how does it look if the provider add-on name is pretended/appended to the channel name?

Far from ideal but might be an acceptable as a starting point.

dagwieers commented 3 years ago

That was one of the options indeed, however some add-on names are rather long (even the add-on id MSB).

Would be nice if we had superscript of subscript BB-codes for this ;-)

matthuisman commented 3 years ago

iptv merge by default adds the add-on name to the channels groups. So users can go "Groups > Add-on Name" to view the channels for that add-on. That can be changed / turned off.

If you did above, in the add-ons themselves you could have a shortcut to view the channels for that add-on xbmc.executebuiltin('ActivateWindow(TVChannels,pvr://channels/tv/addon_name)') (above is kodi 19 only - https://kodi.wiki/view/Opening_Windows_and_Dialogs)

I'm not too sure if many users will care where (what addon) the channel comes from? As long as they can see the name / number and epg. They can always hide duplicates etc via Kodi channel manager? eg. if they wanted Channel A from Add-on A instead of Add-on B. In saying that, no way to tell in channel manager which is which...

I personally wouldn't like the add-on name in the channel name. It's just going to look a bit messy I think? At least have a setting to disable it :)

Also, would it just be a skin change to show a channels groups somewhere? Maybe below the description? Or does kodi not provide that information to skinners?

Also, hope you don't mind - i've been pointing a few add-on devs to your integration page. Much nicer than each trying to make their own IPTV Simple integration.

phunkyfish commented 3 years ago

Currently that information is not provided to skinners. Although I think we might be trying to make a concept fit that doesn’t. Groups can mean many things. For instance I have several groups which split the same source of channels in genres. If a user has another PVR add-on and iptvsimple it can get very messy with groups quickly.

Conceptually would it make more sense for a channel to have a provider field possibly?

matthuisman commented 3 years ago

Provider makes a lot of sense to me. eg. TvHeadend, MythTV, Addon A, Addon B.. Leave groups for categories.

Could it just be a new property? Then we could use #KODIPROP in the playlists and no new code would be required for IPTV Simple as that already parses #KODIPROP.

#EXTINF:-1 group-title="Drama" tvg-chno="1" tvg-id="1",Channel 1
#KODIPROP:pvr.provider=My Addon
plugin://my.video.addon/?_=play&id=1
phunkyfish commented 3 years ago

In IPTV Simple it’s easy. But it would require an API change in kodi which means targeting kodi 20.

dagwieers commented 3 years ago

@matthuisman If the provider would be an add-on id, the implementation could use the add-on logo or add-on name as they see fit.

matthuisman commented 3 years ago

@dagwieers thats a good idea. I wasn't sure if skinners have methods to get add-on info (name / logo) from just the addon id?

dagwieers commented 3 years ago

@dagwieers thats a good idea. I wasn't sure if skinners have methods to get add-on info (name / logo) from just the addon id?

If not, they could be expanded by Kodi so skinners have them.

phunkyfish commented 3 years ago

I will look at adding support for providers in kodi 20. A provider can have a name, a type (thinking satellite, cable, aerial and iptv) and an icon. Anything else that would be needed?

michaelarnauts commented 3 years ago

I will look at adding support for providers in kodi 20. A provider can have a name, a type (thinking satellite, cable, aerial and iptv) and an icon. Anything else that would be needed?

Sounds nice, maybe a Country? Although this might be more something that should be specified on a Channel.

phunkyfish commented 3 years ago

I thought of country but then for providers that span countries it’s not so useful. Language possibly.

Edit: but I guess by default it could have not specified for either language or country.