Skip to content

fix: honor Conversation.visualize options - #1331

Open
avinashgola wants to merge 1 commit into
fishaudio:mainfrom
avinashgola:fix/1325-visualize-options
Open

fix: honor Conversation.visualize options#1331
avinashgola wants to merge 1 commit into
fishaudio:mainfrom
avinashgola:fix/1325-visualize-options

Conversation

@avinashgola

@avinashgola avinashgola commented Sep 5, 2026

Copy link
Copy Markdown

Summary

  • forward use_color and merge_audio_tokens from Conversation.visualize()
  • disable ANSI escape sequences when use_color=False
  • collapse consecutive audio-pad tokens while preserving loss-label boundaries
  • add focused regression tests for both options

Testing

  • python -m pytest tests/test_visualization.py -v (3 passed)
  • ruff check fish_speech/content_sequence.py tests/test_visualization.py
  • ruff check --ignore F401 fish_speech/conversation.py
  • ruff format --check fish_speech/content_sequence.py fish_speech/conversation.py tests/test_visualization.py
  • git diff --check

Fixes #1325


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

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.

Conversation.visualize exposes unsupported use_color and merge_audio_tokens parameters

1 participant