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

Mycroft lost control of playing stream under specific circumstances #65

Open
ChanceNCounter opened this issue Apr 28, 2021 · 4 comments
Labels
bug Something isn't working enhancement New feature or request Priority: Medium

Comments

@ChanceNCounter
Copy link

A few minutes ago, while testing which skills would capture certain requests, I did the following:

  • Start a stream, confirm that the intent is captured and playback is initiated
  • Issue "stop playback" immediately, before the stream had actually commenced
  • Attempt to play a different station, which did not succeed

At this point, the first station started playing, but Mycroft doesn't seem to be in control of the player. It isn't displaying the media player screen, and "stop playback" doesn't work. I'm having trouble even finding the process to kill it.

@ChanceNCounter ChanceNCounter added the bug Something isn't working label Apr 28, 2021
@ChanceNCounter
Copy link
Author

Stopping and starting Mycroft did the trick, so at least we know it was still session-bound XD

@krisgesling
Copy link
Collaborator

If I'm understanding the scenario correctly, I believe this comes back to the need for a more integrated audio and playback service. Spec doc in progress here:
https://docs.google.com/document/d/1ebS60QLna4_AOQxQYpoaZrpkBzPXjL-7VFPJ7-xVvIQ/edit?usp=sharing

Currently there's different ways that audio can be output, and mycroft-core is not necessarily aware of when that happens which I think is an issue.

Which Skill was this in particular?

@krisgesling krisgesling added bug Something isn't working enhancement New feature or request Priority: Medium and removed bug Something isn't working labels May 11, 2021
@ChanceNCounter
Copy link
Author

Off the top of my head, I'm not sure, except that it must have been one of johnbart's radio skills, because I was fussing with how to trigger a couple local stations and their inconsistent keywords.

I'll poke through logs tomorrow, see if I can find it by keyword, but I'm not hopeful.

@ChanceNCounter
Copy link
Author

Alright. It looks like the IHeartRadio skill tied for, was selected for, and processed "play KEXP" at 19:37:27, and was also chosen from a tie for "710 ESPN" at 19:37:43.

The skill then attempted to do this, several times:

self.gui["previous1Title"] = trackHistory_obj["data"][0]["title"]

and gets IndexError: list index out of range

Hopefully soon I'll get the opportunity to paste more of the traceback - uncooperative multiplexer on this machine - but it's all just the skill working its way up to that call.

On the one hand, unlisted skill 🤷‍♂️ but, on the other hand, it clearly comes back to trying to interact with Mycroft's "now playing" features while they're still processing a previous request.

I think the discovery here may be the need for some kind of lock and queue on that stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request Priority: Medium
Projects
None yet
Development

No branches or pull requests

2 participants