Add training Task Type#52
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. To trigger a review, include ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches✨ Simplify code
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Runware/Runware-base.ts`:
- Around line 1093-1095: The call to pollForAsyncResults<TTrainingResponse> is
missing a dedupe key so successful training responses never register with
getUniqueUUID; update the call in Runware-base (the return statement using
pollForAsyncResults) to provide a dedupeKey (or extractor) that yields the same
identifier getUniqueUUID expects (e.g., use request.taskUUID if that is the
training job id, or provide a function that reads the training response's
job/model id field and returns it). Ensure the dedupeKey aligns with
getUniqueUUID logic so successful TTrainingResponse objects are counted and the
poll resolves instead of timing out.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 3150ba41-3019-415b-a466-0620335af684
📒 Files selected for processing (4)
Runware/Runware-base.tsRunware/types.tspackage.jsonreadme.md
0e2e515 to
931464c
Compare
No description provided.