Skip to content

fix: replace conda playsound with pypi playsound3 - #1092

Open
Flova wants to merge 3 commits into
mainfrom
fix/playsound3-dependency
Open

Flova wants to merge 3 commits into
mainfrom
fix/playsound3-dependency

Conversation

@Flova

@Flova Flova commented Oct 10, 2026 •

Copy link
Copy Markdown
Member

The code imports playsound3, but only the unmaintained playsound package was declared and playsound3 is not available on the conda channels, causing a dependency error at runtime.

Fixes #778

Checklist

  • Run pixi run build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

The code imports playsound3, but only the unmaintained playsound package
was declared and playsound3 is not available on the conda channels,
causing a dependency error at runtime.
@MegaIng

MegaIng commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

Does this work on the robot?

@Flova

Flova commented Oct 10, 2026

Copy link
Copy Markdown
Member Author

No tested

@Flova

Flova commented Oct 10, 2026

Copy link
Copy Markdown
Member Author

But at least it installs the lib that the code uses.

@MegaIng

MegaIng commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

Having the PR open is a good reminder to test this ;-)

@MegaIng

MegaIng commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

With the addition of prefer_backend('ffplay') this change now works on the CLs. I haven't been able to test on the real robot because I couldn't get them to produce sound of any kind at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Playsound dependency error

2 participants