Skip to content

Rebased ASR for LFM2-Audio-1.5B#58

Closed
tdakhran wants to merge 3 commits intongxson:xsn/mtmd_refactor_audio_preprocfrom
Liquid4All:tarek/feat/lfm2_asr_rebased
Closed

Rebased ASR for LFM2-Audio-1.5B#58
tdakhran wants to merge 3 commits intongxson:xsn/mtmd_refactor_audio_preprocfrom
Liquid4All:tarek/feat/lfm2_asr_rebased

Conversation

@tdakhran
Copy link
Copy Markdown

No description provided.

@tdakhran tdakhran requested a review from ngxson as a code owner December 15, 2025 10:51
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 15, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment thread tools/mtmd/mtmd-audio.cpp Outdated
Copy link
Copy Markdown
Owner

@ngxson ngxson left a comment

Choose a reason for hiding this comment

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

Looking good overall, just a small comment about preprocessing need to be a dedicated class

@tdakhran tdakhran force-pushed the tarek/feat/lfm2_asr_rebased branch from 2c56532 to 50597aa Compare December 15, 2025 12:18
@tdakhran
Copy link
Copy Markdown
Author

pushed into ggml-org#17694

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants