[black_eagle]

Thanks JB

I did ask in the AS thread if it was possible to pass in artist and/or album data to it but apparently it isn't.

However, given that this is now working well, I think in the new year I make take a look at AS and see if I can hack it around to accept info supplied by the streaming helper. No idea yet if it'll be possible though.

Merry Christmas to you and yours

[black_eagle]

So, I looked into this earlier and actually it was quite easy to add the required stuff

I forked the original code on github and modified it so that if you are listening to an internet stream, it gets the artist name(s) from the properties set by the streaming radio helper.
Note - these properties have been updated in the latest version to be more in-line with other add-on window properties. Please see the updated README on github for what you will need to change in your skin.

Link for the modified artistslideshow is here. I have done some initial testing with it, however further testing is required * looking at you @JohnyBee

Multiple artists needs testing properly - I have added some extra splitting to the standard AS splitting which was just 'feat' or 'ft'. It should now split on ', ', 'vs ' and 'and ' if not followed by 'the' as well. Hopefully this will work OK as srh supplies the full artist list in the srh.Artist property if it detects multiple artists. Artist splitting is similar (but not exactly the same) as the way srh splits artists.

Oh, work-in-progress has now been merged into the master branch with a slight version bump so that is the one you should use JB

I would also be interested in seeing how your MusicVisualisation.xml looks. I need to update the version of mine that I have on github as the one there is terribly out of date compared to the one I am running now.

[JohnyBee]

So, I looked into this earlier and actually it was quite easy to add the required stuff

I forked the original code on github and modified it so that if you are listening to an internet stream, it gets the artist name(s) from the properties set by the streaming radio helper.
Note - these properties have been updated in the latest version to be more in-line with other add-on window properties. Please see the updated README on github for what you will need to change in your skin.

Thanks.
I'm testing the latest version.
When I viewing the visualization window is always automatically opens the settings window for SRH

Link for the modified artistslideshow is here. I have done some initial testing with it, however further testing is required * looking at you @JohnyBee

Look at these posts, please:
link1
link2

Multiple artists needs testing properly - I have added some extra splitting to the standard AS splitting which was just 'feat' or 'ft'. It should now split on ', ', 'vs ' and 'and ' if not followed by 'the' as well. Hopefully this will work OK as srh supplies the full artist list in the srh.Artist property if it detects multiple artists. Artist splitting is similar (but not exactly the same) as the way srh splits artists.

I also created a mod AS, which downloads logos and baners for playing artist. (Author AS dont want to add this to official version).
But AS update to the new version and I do not want to constantly update code.

Oh, work-in-progress has now been merged into the master branch with a slight version bump so that is the one you should use JB

I would also be interested in seeing how your MusicVisualisation.xml looks. I need to update the version of mine that I have on github as the one there is terribly out of date compared to the one I am running now.

My MusicVisualisation.xml is rebuilt to the latest version of SRH , AS, Skin Helper Service and MOD DialogSongInfo for MQ7 and JB-SimilarArtist in music visualization.
My screen for testing:

[black_eagle]

OK, the settings window should only open if you have not set the path to your local music library in the settings for the addon.

I looked at your questions in the AS thread that you linked to. SRH caches MBID's for artists but in any case, by the time my modded version of AS gets the artist name(s) then srh has already checked the MBID and that the name is correct. Maybe I can add the fanart.tv functionality back using MBID's supplied by srh. I'll take a look when I get time

Screenshot looks good

[JohnyBee]

Is SRH case sensitive (e.g feat, Feat, FEAT and fEAt.......)?

[black_eagle]

Is SRH case sensitive (e.g feat, Feat, FEAT and fEAt.......)?

Not currently, no.

[nono123]

Hi,
i glad a found this, because we play a lot of radio on kodi.

But the CD cover show is emty.
There is no radio-logo and no artist-image
But i have everything else.

So i think i missing something?
I installeed the mod and replaced the MusicVisualisation.xml

My specs:
-Rpi3
-Openelec
MQ aeon7

EDIT: logfile = https://nono123.home.xs4all.nl/kodi.log


Thanks, NoNo

[nono123]

I think it found the radio-station logo.
It look like it is behind the original kodi music logo.

[black_eagle]

Hi nono !!

Yes, that picture does indeed suggest that the logo is behind the default music cover. What happens if you use layout 1 ? Does it display correctly then ? I may have got the ordering of layers wrong in the other layouts!

Artist images are supplied by ArtistSlideshow, apart from artist thumbs and banners which are supplied by this add-on (if they exist).

[nono123]

Hi black_eagle,

I was using Layout 3.
Layout 1 seems to work ok here !!
It is layout 2 & 3 where the images from the cover and the discart are switched.

I can not say how i love this addon !!!! We just had the TOP2000 over here.
Very nice to see all the faces behind the music !!!! Even some pics of DJ`s and radio-studio`s, very nice.

Thank you very much.


Is there an option to switch between the radio-station logo and the artist-image ??

[black_eagle]

Hmmmm.....Been a while !!

Some updates in the development branch on github.

Development version now has options in the settings to reverse the track and artist order (Some stations seem to have changed the order they transmit their data in recently). Also, the dev version obtains track info from either TADB or last.fm and sets it as a property (srh.TrackInfo).

Also includes a few bugfixes and better error handling when TADB is not available.

Is there an option to switch between the radio-station logo and the artist-image ??

Not currently. That's handled by the skin modifications. Specifically, it looks for the station logo (as supplied by the rad.io addon), then in your music collection for a directory named the same as the radio station (you can drop a 'cover.png' into there for a custom cover), then for an artist thumbnail and then for an actual CD cover (if the addon matches a track to an album and that album is in your collection it'll display that cover) and then it falls back to the default. These are all stacked one on top of another so the last one it finds is the one you see. This means if there is a an album cover in your collection and a station logo, the album cover will be displayed in preference to the station logo. If you change the order in the skin files its possible to change what gets displayed. I might be able to look into providing a setting in the skin to change the displayed image. Probably if I can do this it'll go in the view menu (where you select from view 1,2,3 etc).

In the meantime, I'll look into what is/was going on with the other views and get an updated version up onto github.

EDIT

The easiest way to add the trackinfo is to edit the 'includesvariables.xml' file in MQ7. Find the '<variable name="value_playerplot">' in the file (in my case its line 2350) and add [CR] $INFO[Window(Visualisation).Property(srh.TrackInfo),[CR] - ] after '$INFO[Window(Visualisation).Property(ArtistSlideshow.5.SimilarName), / ]'. Save the file and reload the skin. Any track info available should now appear (in view 1) in the text box before the artist info.

[sudden]

Hi black_eagle!
On last Krypton no metadata on radio streams as before... The problem has not been fixed so far.
I use PVR IPTV Simple Client with m3u playlist for various radio streams.Tested on many skins. Why ticket #16948 is closed? I'm so confused

[svortevik]

I think it found the radio-station logo.
It look like it is behind the original kodi music logo.

Did you figure this out?

I have the same problem on the picture :/

I am also missing the artist logo.

How do I change layout?

[docbrown0621]

I am running Kodi v17 on Raspbian Stretch using the carpc-xtouch skin https://www.tapatalk.com/groups/engineer...-t633.html.  It looks like all is working fine in the logs but I don't get the album art or info on the screen.  Have you, or anyone else been successful with this skin?

[black_eagle]

I am running Kodi v17 on Raspbian Stretch using the carpc-xtouch skin https://www.tapatalk.com/groups/engineer...-t633.html.  It looks like all is working fine in the logs but I don't get the album art or info on the screen.  Have you, or anyone else been successful with this skin?

 The radio helper just sets some skin properties that can be displayed by any skin as long as it is modified (or already supports) displaying that info.  Whether this is the case with carpc-xtouch I don't know, but if the author included support for this addon, Iwould expect it to work.  If not, you will have to modify the skin yourself to get the info - details are at https://github.com/the-black-eagle/scrip...dio.helper - NOTE, info is for Kodi v18 but should be reasonably backward-compatible with v17.

If compatibility is not included with the skin, perhaps I can help, but no guarantee's  as I do not have a local copy of v17.