Fix GetTasksForQueueAsync

[johnml1135]

• Speed up NmtQueue test with parallel corpora
• ClearML queue position should include all items, not just the ones applicable to the engine type #513

---

This change is 

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 56.95%. Comparing base (e2f1f25) to head (6069efc).

Files with missing lines	Patch %	Lines
...c/Serval.Machine.Shared/Services/ClearMLService.cs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #534   +/-   ##
=======================================
  Coverage   56.95%   56.95%           
=======================================
  Files         299      299           
  Lines       15734    15734           
  Branches     2173     2173           
=======================================
  Hits         8962     8962           
  Misses       6122     6122           
  Partials      650      650           

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

[ddaspit]

Reviewed 3 of 3 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @Enkidu93 and @johnml1135)

---

src/Machine/src/Serval.Machine.Shared/Services/ClearMLService.cs line 165 at r1 (raw file):

        var tasks = (JsonArray?)result?["data"]?["queue"]?["entries"];
        IEnumerable<string> taskIds = tasks?.Select(t => (string)t?["task"]!) ?? new List<string>();
        var tasksById = await GetTasksByIdAsync(taskIds, cancellationToken);

I'm guessing this change was made for debugging purposes. It can probably be reverted.

---

src/Serval/test/Serval.E2ETests/ServalApiTests.cs line 218 at r1 (raw file):

        Assert.That(engine.IsModelPersisted, Is.True);
        string[] books = ["bible_LARGEFILE.txt"];
        var train_corpus = await _helperClient.MakeParallelTextCorpus(books, "es", "en", false);

Please use an explict type if the type is specified elsewhere on the line.

[johnml1135]

src/Machine/src/Serval.Machine.Shared/Services/ClearMLService.cs line 165 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

I'm guessing this change was made for debugging purposes. It can probably be reverted.

Done.

[johnml1135]

src/Serval/test/Serval.E2ETests/ServalApiTests.cs line 218 at r1 (raw file):

Previously, ddaspit (Damien Daspit) wrote…

Please use an explict type if the type is specified elsewhere on the line.

Done.

[ddaspit]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Enkidu93)