Add sequence confidence to pretranslations

[Enkidu93]

Fixes #268, #271

See also sillsdev/silnlp#966 which came out of this work.

---

This change is 

[codecov-commenter]

Codecov Report

❌ Patch coverage is 93.33333% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.49%. Comparing base (b9219fc) to head (64ecec9).

Files with missing lines	Patch %	Lines
...translation/huggingface/hugging_face_nmt_engine.py	90.90%	2 Missing ⚠️
machine/translation/translation_result_builder.py	83.33%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #279   +/-   ##
=======================================
  Coverage   91.49%   91.49%           
=======================================
  Files         355      355           
  Lines       22801    22829   +28     
=======================================
+ Hits        20862    20888   +26     
- Misses       1939     1941    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ddaspit]

@ddaspit reviewed 9 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on Enkidu93 and mshannon-sil).

---

machine/translation/translation_result.py line 55 at r1 (raw file):

    @property
    def sequence_confidence(self) -> Optional[float]:

We should have a default value, so that the confidence isn't optional, i.e. -1.

[Enkidu93]

@Enkidu93 made 1 comment.
Reviewable status: 5 of 9 files reviewed, 1 unresolved discussion (waiting on ddaspit and mshannon-sil).

---

machine/translation/translation_result.py line 55 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

We should have a default value, so that the confidence isn't optional, i.e. -1.

Done. I moved the default value to the builder because we exponentiate the value when using the builder to create the result and it seemed a but cleaner this way. Would you prefer it here or in the builder?

[ddaspit]

@ddaspit reviewed 4 files and all commit messages, made 2 comments, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on mshannon-sil).

---

machine/translation/translation_result.py line 55 at r1 (raw file):

Previously, Enkidu93 (Eli C. Lowry) wrote…

Done. I moved the default value to the builder because we exponentiate the value when using the builder to create the result and it seemed a but cleaner this way. Would you prefer it here or in the builder?

This is good.