Skip to content

Align remove_bg params with the PhotoRoom API. - #347

Open
ckcnik wants to merge 1 commit into
mainfrom
UCCORE-remove-bg-params
Open

ckcnik wants to merge 1 commit into
mainfrom
UCCORE-remove-bg-params

Conversation

@ckcnik

@ckcnik ckcnik commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Typed execute params are crop, bg_color, and channels so clients send the current REST body instead of the old remove.bg fields.

Description

Checklist

Typed execute params are crop, bg_color, and channels so clients send the current REST body instead of the old remove.bg fields.

Co-authored-by: Cursor <cursoragent@cursor.com>
@sonarqubecloud

Copy link
Copy Markdown

Comment thread HISTORY.md

- **Breaking**: `AddonRemoveBGExecutionParams` now matches the PhotoRoom API
(`crop`, `bg_color`, and `channels`). The old remove.bg fields are no longer
accepted.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The same question regarding provider we had discussed today. Let's remove provider's references?

Another point:
I suggest to rewrite this to something like:

(`crop_margin`, `scale`, `add_shadow`, `type_level`, `type`, `semitransparency`, `roi`, `position`) are removed with no replacement.
``` - print list of parameters and mention about no other replacement of them

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