Repository navigation
[codex] docs: complete translation configuration reference - #2132
Conversation
Signed-off-by: Lawrence Lane <llane@nvidia.com>
| - `backend_type="llm"` requires both `client` and a non-empty `model_name`, even in dry-run mode. | ||
| - FAITH always uses an LLM client. With Google, AWS, NMT, or a custom translation backend, pass a separate `client` and set `faith_model_name`. | ||
| - `merge_scores=True` with `output_mode="replaced"` is rejected. With FAITH disabled, score merging is skipped with a warning. | ||
| - `output_mode="raw"` removes `output_field` after building `translation_metadata`. Do not combine raw mode with message reconstruction; use `"both"` when you need metadata and `translated_messages`. |
There was a problem hiding this comment.
The guidance correctly says not to combine
output_mode="raw" with reconstruct_messages, but doesn't explain the concrete consequence. In FormatTranslationOutputStage.process, output_field is dropped from the DataFrame before _build_translated_messages is called, so every translated_messages entry ends up as an empty string. Describing that failure mode here will save users from a confusing silent-empty result.
| - `output_mode="raw"` removes `output_field` after building `translation_metadata`. Do not combine raw mode with message reconstruction; use `"both"` when you need metadata and `translated_messages`. | |
| - `output_mode="raw"` removes `output_field` after building `translation_metadata`. Do not combine raw mode with message reconstruction: `output_field` is dropped before `_build_translated_messages` runs, so every `translated_messages` entry will be an empty string. Use `output_mode="both"` when you need metadata and `translated_messages`. |
Signed-off-by: Lawrence Lane <llane@nvidia.com>
Signed-off-by: Lawrence Lane <llane@nvidia.com>
|
🌿 Preview your docs: https://nvidia-preview-codex-docs-translation-reference.docs.buildwithfern.com/nemo/curator Here are the markdown pages you've updated: |
|
@ayushdg friendly ping on this docs PR for the #2118 release-docs workstream — when you have a moment, could you take a look at the translation configuration reference? Docs preview: https://nvidia-preview-codex-docs-translation-reference.docs.buildwithfern.com/nemo/curator Thanks! |
|
@sarahyurick, since you approved the underlying translation-controls implementation in #2038, could you please review this configuration reference when you have a moment? Thank you! |
sarahyurick
left a comment
There was a problem hiding this comment.
I don't know so much about this pipeline, but just from eyeballing these changes look fine to me.
|
/ok to test e9d0c7e |
…Mo#2132) * docs: complete translation configuration reference Signed-off-by: Lawrence Lane <llane@nvidia.com> * docs: clarify raw translation output Signed-off-by: Lawrence Lane <llane@nvidia.com> * docs: clarify FAITH client concurrency Signed-off-by: Lawrence Lane <llane@nvidia.com> --------- Signed-off-by: Lawrence Lane <llane@nvidia.com>
What changed
TranslationStageparameter reference covering inputs, segmentation, inference, FAITH evaluation, output, and resume controlsWhy
The translation controls introduced in #2038 were not fully represented in the user guide. Users had to inspect implementation details to discover defaults, constraints, backend requirements, and the behavior of resumed translation runs.
Impact
Users can configure and validate experimental translation pipelines directly from the guide, including mixed-provider translation and quality evaluation workflows. This is a documentation-only change; runtime behavior is unchanged.
Validation
cd fern && npm run check— passes with 0 errorscd fern && fern docs broken-links— changed page has no broken links; the command reports 22 pre-existing errors in older API-reference pagesgit diff --check— passesCloses #2126
Parent workstream: #2118