mopidy-tidal icon indicating copy to clipboard operation
mopidy-tidal copied to clipboard

Mopidy 4.0 is going to be released

Open fmarzocca opened this issue 1 year ago • 6 comments

As per title... did you check this Changelog: https://docs.mopidy.com/en/develop/changelog/#v4-0-0-unreleased ? TlTracks objects are removed for Mopidy Core.

fmarzocca avatar Feb 26 '24 09:02 fmarzocca

Thanks for the heads-up. Not sure if there are any breaking changes but feel free to share your findings if/when you test with v4.0

tehkillerbee avatar Feb 29 '24 23:02 tehkillerbee

On 4.0 I get

Dec 28 20:39:36 home mopidy[3342027]:   Starting playback of track:265699083, (quality:HIGH, codec:MP4A, 16bit/44100Hz)
Dec 28 20:39:36 home mopidy[3342027]: ERROR    2024-12-28 20:39:36,405 [3342027:MainThread] mopidy.audio.gst
Dec 28 20:39:36 home mopidy[3342027]:   GStreamer error: Invalid URI "['https://amz-pr-fa.audio.tidal.com/3f87aca4f573bc761fe611498e6c4dbc_37.mp4?token=1735418376~NTA2OWUwOGIxNWMzZTQ5ZjRmZjU1MWVjYTUyZDQyNjlmNDgxNmRiYg==']"

with HIGH or with LOSSLESS

Dec 28 19:10:15 home mopidy[3306278]:   Starting playback of track:57269960, (quality:LOSSLESS, codec:FLAC, 16bit/44100Hz)
Dec 28 19:10:15 home mopidy[3306278]: ERROR    2024-12-28 19:10:15,244 [3306278:MainThread] mopidy.audio.gst
Dec 28 19:10:15 home mopidy[3306278]:   GStreamer error: Invalid URI "['https://sp-pr-fa.audio.tidal.com/mediatracks/CAEaKwgDEic4Zjc1MmMzMmRiZmQ1ODg4NjdkNzkzNjQ3Zjk2ZjM3N182MS5tcDQ/0.flac?token=1735413015~NGI1MTI3MzgxYzdkZGVhYmMxYjg4OWI1>
Dec 28 19:10:15 home mopidy[3306278]: WARNING  2024-12-28 19:10:15,245 [3306278:Audio-2 (_actor_loop)] mopidy.audio.actor
Dec 28 19:10:15 home mopidy[3306278]:   Setting GStreamer state to GST_STATE_PLAYING failed

And sure, gst-plugins-good, bad and ugly are installed.

bjo81 avatar Dec 28 '24 19:12 bjo81

This sounds a lot like https://github.com/tehkillerbee/mopidy-tidal/issues/185.

Have you installed the latest version of mopidy-tidal?

tehkillerbee avatar Dec 28 '24 19:12 tehkillerbee

Grmpf, seems like it's outdated on Arch's AUR.

bjo81 avatar Dec 28 '24 19:12 bjo81

Thanks for the hint, updating to 0.3.9 fixed it.

bjo81 avatar Dec 28 '24 19:12 bjo81

Ok I've found some time to get the library lookup methods to work also with the new Mopidy conventions: https://github.com/tehkillerbee/mopidy-tidal/pull/196.

Note that in that PR I'm striking to maintain back-compatibility (all the dirty conversion logic happens in _convert_tracks), so Mopidy 3.x should still be supported.

So far the use cases I have tested are quite limited (adding a playlist and adding a track) because playback is broken on all of my Arch Linux systems because of GStreamer issues (see https://github.com/mopidy/mopidy/issues/2202, https://github.com/mopidy/mopidy/issues/2180 and https://github.com/mopidy/mopidy/issues/2183), so feel free to give it a try and report back if you find any issues.

blacklight avatar Jul 05 '25 19:07 blacklight