Skip to content

add modality-aware Instructions with audio/text variants#4987

Open
longcw wants to merge 5 commits intomainfrom
longc/modality-aware-instructions
Open

add modality-aware Instructions with audio/text variants#4987
longcw wants to merge 5 commits intomainfrom
longc/modality-aware-instructions

Conversation

@longcw
Copy link
Contributor

@longcw longcw commented Mar 3, 2026

No description provided.

@chenghao-mou chenghao-mou requested a review from a team March 3, 2026 14:49
devin-ai-integration[bot]

This comment was marked as resolved.

Copy link
Member

@chenghao-mou chenghao-mou left a comment

Choose a reason for hiding this comment

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

LGTM. Tested locally with different modalities.

"""
return self._text_variant if self._text_variant is not None else self._audio_variant

def for_modality(self, modality: Literal["audio", "text"]) -> Instructions:
Copy link
Member

Choose a reason for hiding this comment

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

nit: should we use to_modality, similar to our to_provider_format?

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.

2 participants