Skip to content

Feat/new train request endpoint#555

Merged
kitsudaiki merged 4 commits intodevelopfrom
feat/new-train-request-endpoint
May 17, 2025
Merged

Feat/new train request endpoint#555
kitsudaiki merged 4 commits intodevelopfrom
feat/new-train-request-endpoint

Conversation

@kitsudaiki
Copy link
Copy Markdown
Owner

Pull Request

Description

Add train- and request-api-endpoint

Related Issues

How it was tested?

  • updated sdk-api-tests

A new switch for the cluster was added to change the
mode between direct and task mode again. The new
implementation is also more robust than the old one.

Signed-off-by: Tobias Anker <tobias.anker@kitsunemimi.moe>
Added new train and request api endpoint to directly
trigger these actions for single-inputs without an
async task.
Also tested the request in the sdk-api-test to check
the train results.

Signed-off-by: Tobias Anker <tobias.anker@kitsunemimi.moe>
Cluster-handle are not not locked per task, but per
iteration within the task. This way, tasks and direct
interaction can be easily mixed and doesn't require a
switch of the mode anymore.

Signed-off-by: Tobias Anker <tobias.anker@kitsunemimi.moe>
The cluster-mode, which was an original idea of the old
backend c++ implmentation, was removed again. While
implementing the train and request endpoint for direct
interation, i found out, that the new workflow of the
cluster provides more oportunities. So in the new version
the direct-input can be mixed with the task-input without
any conflicts.

Signed-off-by: Tobias Anker <tobias.anker@kitsunemimi.moe>
@kitsudaiki kitsudaiki self-assigned this May 17, 2025
@kitsudaiki kitsudaiki merged commit c583c4f into develop May 17, 2025
28 checks passed
@kitsudaiki kitsudaiki deleted the feat/new-train-request-endpoint branch May 17, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant