Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Do not set compute type#511

Open
pierotofy wants to merge 3 commits intoargosopentech:masterfrom
pierotofy:nocompute
Open

Do not set compute type#511
pierotofy wants to merge 3 commits intoargosopentech:masterfrom
pierotofy:nocompute

Conversation

@pierotofy
Copy link
Contributor

@pierotofypierotofy commentedFeb 5, 2026
edited
Loading

This PR fixed a problem introduced with5a9ec1e which broke support for certain models (e.g. Spanish).

Ref:LibreTranslate/LibreTranslate#930

The root cause is a quirky behavior of ctranslate2 which I remember encountering in#369, whereby setting the compute_type explicitly would cause certain models to return garbage.

Removing the explicit compute_type override and letting ctranslate2 figure out the appropriate value fixes the problem and restores the behavior of v1.9.6.

[piero:~/Documents/argos-translate] [venv] master* ± ARGOS_DEBUG=1 ARGOS_CHUNK_TYPE=STANZA python test_trans.py('get_installed_languages',)('paragraphs:', ['Vamos'])('apply_packaged_translation','Vamos')('Splitting sentences using SBD Model: (es) StanzaSentencizer',)('sentences', ['Vamos'])('tokenized', [['Vamos']])('translated_batches', [TranslationResult(hypotheses=[['mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@', 'mainstre@@']], scores=[-39.73262023925781], attention=[], logits=[])])('value_hypotheses:', [('mainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstre@@', -39.73262023925781)])('translated_paragraphs:', [[('mainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstre@@', -39.73262023925781)]])('hypotheses_to_return:', [('mainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstre@@', -39.73262023925781)])mainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstremainstre@@[piero:~/Documents/argos-translate] [venv] master* ± git commit -a -m"Revert 5a9ec1e"[master 13c2130] Revert 5a9ec1e 2 files changed, 2 deletions(-) [piero:~/Documents/argos-translate] [venv] master(1)* ± ARGOS_DEBUG=1 ARGOS_CHUNK_TYPE=STANZA python test_trans.py('get_installed_languages',)('paragraphs:', ['Vamos'])('apply_packaged_translation','Vamos')('Splitting sentences using SBD Model: (es) StanzaSentencizer',)('sentences', ['Vamos'])('tokenized', [['Vamos']])('translated_batches', [TranslationResult(hypotheses=[['Come','on','.']], scores=[-0.7667394280433655], attention=[], logits=[])])('value_hypotheses:', [('Come on.', -0.7667394280433655)])('translated_paragraphs:', [[('Come on.', -0.7667394280433655)]])('hypotheses_to_return:', [('Come on.', -0.7667394280433655)])Come on.

@pierotofy
Copy link
ContributorAuthor

Edited to allow compute_type to be overriden, but don't set it if it'sauto.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@pierotofy

Comments


[8]ページ先頭

©2009-2026 Movatter.jp