Skip to content

Added new Ultralytics param inpaintSize#299

Open
Sirsho1997 wants to merge 2 commits into
mainfrom
feature-ultralyticsInpaintSize
Open

Added new Ultralytics param inpaintSize#299
Sirsho1997 wants to merge 2 commits into
mainfrom
feature-ultralyticsInpaintSize

Conversation

@Sirsho1997
Copy link
Copy Markdown
Collaborator

@Sirsho1997 Sirsho1997 commented May 25, 2026

Added

  • IUltralytics now includes:
    • inpaintSize: Optional[int]

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

Important

Review skipped

Auto reviews are disabled on this repository. To trigger a review, include review in the PR description. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f085908e-bafd-4985-8a16-fd6cb5a2a95b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feature-ultralyticsInpaintSize

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new optional inpaintSize field to the IUltralytics dataclass, per the changelog note for 0.5.14. However, the PR also modifies the production WebSocket endpoint, which appears unrelated and incorrect.

Changes:

  • Add inpaintSize: Optional[int] to IUltralytics.
  • Change the production WebSocket URL in BASE_RUNWARE_URLS from runware.ai to runware.dev.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
runware/types.py Adds the new inpaintSize optional field to IUltralytics.
runware/utils.py Modifies the production WebSocket URL — appears unrelated to the PR's purpose and likely a regression.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread runware/utils.py Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

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.

2 participants