Separate key terms from other training data on S3

[pmachapman]

Fixes #816. Requires sillsdev/machine.py#254

---

This change is 

[Enkidu93]

@Enkidu93 reviewed 10 of 10 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @pmachapman)

[ddaspit]

@ddaspit reviewed 10 of 10 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @pmachapman)

---

src/ServiceToolkit/src/SIL.ServiceToolkit/Services/IParallelCorpusPreprocessingService.cs line 12 at r1 (raw file):

    Task PreprocessAsync(
        IReadOnlyList<ParallelCorpus> corpora,
        Func<Row, bool, Task> train,

I would prefer an enum here. It allows us to easily support more than two types of data and make it clearer what this parameter means.

[pmachapman]

Reviewable status: 6 of 11 files reviewed, 1 unresolved discussion (waiting on @ddaspit and @Enkidu93)

---

src/ServiceToolkit/src/SIL.ServiceToolkit/Services/IParallelCorpusPreprocessingService.cs line 12 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

I would prefer an enum here. It allows us to easily support more than two types of data and make it clearer what this parameter means.

An enum is a great idea. Please let me know if the enum and its members are named appropriately.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 87.67123% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.11%. Comparing base (564f686) to head (9beab59).

Files with missing lines	Patch %	Lines
...Shared/Services/WordAlignmentPreprocessBuildJob.cs	43.75%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #846      +/-   ##
==========================================
+ Coverage   66.04%   66.11%   +0.06%     
==========================================
  Files         382      382              
  Lines       20635    20696      +61     
  Branches     2700     2706       +6     
==========================================
+ Hits        13629    13683      +54     
- Misses       6043     6050       +7     
  Partials      963      963              

☔ 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.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ddaspit]

@ddaspit reviewed 5 of 5 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @pmachapman)

---

src/ServiceToolkit/src/SIL.ServiceToolkit/Services/IParallelCorpusPreprocessingService.cs line 12 at r1 (raw file):

Previously, pmachapman (Peter Chapman) wrote…

An enum is a great idea. Please let me know if the enum and its members are named appropriately.

Looks good.