Skip to content

Adding vref option to translate command#952

Merged
laura-burdick-sil merged 8 commits intomasterfrom
translate_export_to_vref
Mar 3, 2026
Merged

Adding vref option to translate command#952
laura-burdick-sil merged 8 commits intomasterfrom
translate_export_to_vref

Conversation

@laura-burdick-sil
Copy link
Collaborator

@laura-burdick-sil laura-burdick-sil commented Mar 2, 2026

When running the translate command, this adds an option to output a file in vref format (original versification) along with the SFM file.


This change is Reviewable

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a --vref option to the NMT translate command to export an additional verse-aligned text artifact (original versification) alongside the generated USFM output.

Changes:

  • Introduces convert_usfm_to_vref() to extract a VREF-aligned text stream from a USFM file via machine.corpora.
  • Adds a vref: bool = False option to translate_books() and translate_files() and wires it through CLI --vref.
  • Generates VREF outputs for both single-translation and multi-draft translation outputs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@Enkidu93 Enkidu93 left a comment

Choose a reason for hiding this comment

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

:lgtm:

@Enkidu93 reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on laura-burdick-sil).

@laura-burdick-sil laura-burdick-sil merged commit bb2b13a into master Mar 3, 2026
1 check was pending
@laura-burdick-sil laura-burdick-sil deleted the translate_export_to_vref branch March 3, 2026 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants