Skip to content

refactor(audio): derive sample rate from settings#210

Open
gaspardpetit wants to merge 2 commits intomainfrom
codex/refactor-audio-processing-for-chunk-sizes-1dbad9
Open

refactor(audio): derive sample rate from settings#210
gaspardpetit wants to merge 2 commits intomainfrom
codex/refactor-audio-processing-for-chunk-sizes-1dbad9

Conversation

@gaspardpetit
Copy link
Owner

Summary

  • replace hard-coded 16 kHz conversions in transcript formatting with AUDIO_PARAMS.sample_rate
  • use shared audio settings in voice-processing modules and main pipeline logic
  • update tests to honor configurable sample rate

Testing

  • make check
  • python -m unittest discover -s tests -p 'test_audio*.py' -q
  • python -m unittest discover -s tests -p 'test_stereo_diarization.py' -q
  • python -m unittest discover -s tests -p 'test_transcript*.py' -q
  • ⚠️ python -m unittest discover -s tests -p 'test_window.py' -q (missing torch dependency)

https://chatgpt.com/codex/tasks/task_e_68b5a8e4cbcc832c9d044492632fd0a1

@github-actions
Copy link

github-actions bot commented Sep 1, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  verbatim
  __init__.py
  config.py
  main.py
  verbatim.py 371-376, 555, 609-616
  verbatim/audio
  audio.py
  verbatim/audio/sources
  audiosource.py
  factory.py
  ffmpegfileaudiosource.py
  fileaudiosource.py 33-38
  micaudiosource.py 7-13, 85-89, 151-158
  pcmaudiosource.py 4-10, 71
  wavsink.py 3-20
  verbatim/transcript
  sentences.py
  verbatim/transcript/format
  docx.py 8-17, 77-100
  json.py 29
  md.py 4-12, 128-151
  txt.py 93, 96-105, 108-112
  verbatim/voices
  diarization.py
  isolation.py 20-22, 68-82
  verbatim/voices/diarize
  factory.py
  pyannote.py
  stereo.py
  verbatim/voices/transcribe
  faster_whisper.py
  whisper.py 6-14, 144-149
  whispermlx.py 7-15, 129-134, 141-147, 170
Project Total  

The report is truncated to 25 files out of 30. To see the full report, please visit the workflow summary page.

This report was generated by python-coverage-comment-action

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant