Skip to content

Replace TachyPy audio backends with TachyAudio#11

Merged
mathiassalvas merged 15 commits into
mainfrom
dev_v0.1.16
Jun 15, 2026
Merged

Replace TachyPy audio backends with TachyAudio#11
mathiassalvas merged 15 commits into
mainfrom
dev_v0.1.16

Conversation

@iancharest

Copy link
Copy Markdown
Contributor

Summary

  • Replace TachyPy-owned sounddevice/dummy audio backends with a thin tachyaudio.OutputStream wrapper
  • Add tachyaudio>=0.2.0b1 as the base audio dependency and remove sounddevice extras/docs
  • Raise TachyPy's Python requirement to >=3.10 to match TachyAudio
  • Update audio tests to mock TachyAudio streams instead of sounddevice/dummy backends
  • Bump package metadata to 0.1.17 because 0.1.16 is already published

Validation

  • ./.venv/bin/pytest --cov=tachypy --cov-report=term-missing --cov-fail-under=80 -> 82 passed, 92.04% coverage
  • ./.venv/bin/ruff check src tests -> passed
  • ./.venv/bin/python -m sphinx -b html docs /tmp/tachypy-docs-build -> passed
  • ./.venv/bin/python -m pip check -> no broken requirements
  • Temporary venv install resolved tachyaudio==0.2.0b1 from PyPI and imported tachypy.Audio successfully

Copilot AI review requested due to automatic review settings June 13, 2026 19:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR migrates TachyPy’s Audio utility away from TachyPy-owned sounddevice/dummy backends to a thin wrapper around tachyaudio.OutputStream, aligning packaging, docs, and tests with the new dependency and raising the minimum Python version to match TachyAudio.

Changes:

  • Replace internal audio backend implementations with a tachyaudio.OutputStream-backed scheduler.
  • Update tests to mock tachyaudio streams instead of sounddevice/dummy backends.
  • Update packaging/docs/CI for tachyaudio>=0.2.0b1, Python >=3.10, and remove TACHYPY_AUDIO_BACKEND=dummy CI configuration.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 12 comments.

Show a summary per file
File Description
src/tachypy/audio.py Replaces backend abstraction with direct tachyaudio.OutputStream usage and adds stream configuration options.
tests/test_audio.py Updates unit tests to use the new default backend behavior and error expectations.
tests/test_audio_extended.py Reworks extended tests to validate tachyaudio stream interactions via fakes.
tests/test_audio_backends.py Adjusts tests to reflect tachyaudio being required and legacy backends rejected.
setup.cfg Adds tachyaudio>=0.2.0b1, bumps version, raises Python requirement, updates extras.
setup.py Bumps version and raises python_requires.
requirements.txt Adds tachyaudio>=0.2.0b1.
README.md Updates installation/audio documentation to reflect tachyaudio-backed audio.
docs/audio.rst Rewrites audio docs around TachyAudio-backed scheduling and configuration.
docs/getting_started.rst Updates installation guidance and optional extras list.
docs/testing.rst Updates CI testing guidance to mock tachyaudio.OutputStream.
docs/examples.rst Removes mention of TACHYPY_AUDIO_BACKEND=dummy.
docs/conf.py Switches autodoc mock import from sounddevice to tachyaudio.
.github/workflows/ci.yml Removes dummy-backend env var from CI matrix jobs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/tachypy/audio.py Outdated
Comment thread src/tachypy/audio.py
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md
Comment thread docs/getting_started.rst Outdated
Comment thread docs/audio.rst Outdated
Comment thread README.md Outdated
Comment thread docs/getting_started.rst Outdated
Comment thread setup.cfg Outdated
iancharest and others added 6 commits June 13, 2026 15:23
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@iancharest

Copy link
Copy Markdown
Contributor Author

closes #1

@iancharest iancharest requested a review from mathiassalvas June 13, 2026 19:31
iancharest and others added 2 commits June 13, 2026 15:33
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@mathiassalvas mathiassalvas merged commit 383ac16 into main Jun 15, 2026
16 checks passed

Copy link
Copy Markdown
Contributor Author

Published TachyPy 0.1.17 to PyPI: https://pypi.org/project/tachypy/0.1.17/

Release checks run locally before upload:

  • pytest --cov=tachypy --cov-report=term-missing --cov-fail-under=80 -> 84 passed, 92.20% coverage
  • ruff check src tests -> passed
  • python -m sphinx -b html docs /tmp/tachypy-docs-build -> passed
  • python -m pip check -> passed
  • python -m build --no-isolation --outdir ... + twine check -> passed

PyPI verification now reports latest version as 0.1.17.

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

Successfully merging this pull request may close these issues.

4 participants