Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ID4 HUD Shows artist instead of track #728

Open
dergizzle opened this issue Jul 30, 2023 · 4 comments
Open

ID4 HUD Shows artist instead of track #728

dergizzle opened this issue Jul 30, 2023 · 4 comments

Comments

@dergizzle
Copy link

First of all - thank you for this! After some fiddling and going through old phones I now got a setup that can play via USB and connects more often than not (using the Connected Classic as gateway app instead of Connected Drive 6.4 helped a lot).

Two issues I have noticed, one is already under discussion (Spotify Icon opening car info display).

My setup: 2014 ID4 5 series, Oneplus 2 with OxygenOS 3.6.1 (based on 6.0.1), latest Spotify, USB connection.

When playing music via Spotify the HUD shows the artist instead of the track. I have found no settings to adjust this. Is this intebtional or a bug, any chance to change it or get a way to change the behavior?

(Any way to integegrate a playlist into the HUD like its shown with the native Deezer Integration?)

Thanks!

@hufman
Copy link
Collaborator

hufman commented Jul 30, 2023

Looking in the code, the app only sends the title to the Instrument Cluster. The MultimediaInfoEvent is used to set the ID4 sidebar's two lines of text, and as far as I know it doesn't affect the HUD.
I don't have an ID4 car to test, so I'd need someone with an ID4 car to try changing the code and reproducing and fixing this bug.

@dergizzle
Copy link
Author

Guess I'd have to try my luck setting up the dev environment, never did anything on Android though. Looking at the code and if that is the only place where that info is sent my best guess (without any Insight into any of it) would be that multimediaInfoEvent has them in the wrong order so I'd probably just switch what gets sent to the cluster and see what happens.

@dergizzle
Copy link
Author

dergizzle commented Aug 1, 2023

Looking in the code, the app only sends the title to the Instrument Cluster. The MultimediaInfoEvent is used to set the ID4 sidebar's two lines of text, and as far as I know it doesn't affect the HUD. I don't have an ID4 car to test, so I'd need someone with an ID4 car to try changing the code and reproducing and fixing this bug.

So I managed to get the Dev Environment working and as a first test compiled it with the output simply changed to:

instrumentCluster.getTextModel()?.asRaDataModel()?.value = UnicodeCleaner.clean(song.artist ?: "")

Same output, so they are not just reversed or something. Now, not being a dev, things get a little difficult for me, maybe you have some pointers for me what I could test. Since the song info comes from the controller object, so my next step would be to find out where that gets set up and go on from there... wich is ... where? :)

I wonder if this may be related to Spotify and it's API? Thinking about it I should try another music app and see how the output is looking there ...

@hufman
Copy link
Collaborator

hufman commented Aug 1, 2023

I'm glad you figured out the dev environment, I tried to make the instructions clear :)
That's interesting that changing the instrumentCluster to show the artist didn't update in the car. What happens when you swap the lines of the multimedia event? I had understood them to just be the order shown in the side bar. If it does matter, feel free to send a PR fixing it!
The song data comes relatively directly from the music controller, and so the same song.title that shows up in the main view and the PhoneUI miniplayer will also be available as song.title in GlobalMetadata. You can use Android Studio's navigation-by-reference to help track how the data flows. Of course, the SpotifyAppController is separate from the GenericMusicController used by other music apps, so trying a different music app would be a useful test.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants