Skip to content

Conversation

@ApplEOFDiscord
Copy link
Collaborator

Motivation

  • support processing transparent image

Modifications

Usage or Command

Accuracy Tests

Checklist

  • Add at least a tag in the PR title.
    • Tag list: [[FDConfig],[APIServer],[Engine], [Scheduler], [PD Disaggregation], [Executor], [Graph Optimization], [Speculative Decoding], [RL], [Models], [Quantization], [Loader], [OP], [KVCache], [DataProcessor], [BugFix], [Docs], [CI], [Optimization], [Feature], [Benchmark], [Others], [XPU], [HPU], [GCU], [DCU], [Iluvatar], [Metax]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • Format your code, run pre-commit before commit.
  • Add unit tests. Please write the reason in this PR if no unit tests.
  • Provide accuracy results.
  • If the current PR is submitting to the release branch, make sure the PR has been submitted to the develop branch, then cherry-pick it to the release branch with the [Cherry-Pick] PR tag.

@paddle-bot
Copy link

paddle-bot bot commented Nov 4, 2025

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Nov 4, 2025


def has_transparent_background(img):
"""判断图片是否有背景"""
Copy link
Collaborator

Choose a reason for hiding this comment

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

改成英文注释

img = change_I16_to_L(img)
if has_transparent_background(img):
img = add_white_background(img)
except Exception:
Copy link
Collaborator

Choose a reason for hiding this comment

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

此处建议加一行异常warning日志

Copy link

@Yelrose Yelrose left a comment

Choose a reason for hiding this comment

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

LGTM

@Jiang-Jia-Jun Jiang-Jia-Jun merged commit 131d76d into PaddlePaddle:develop Nov 5, 2025
17 of 23 checks passed
ApplEOFDiscord added a commit to ApplEOFDiscord/FastDeploy that referenced this pull request Nov 6, 2025
* process transparent image

* english comments

* process transparency at downloading

* fix

* remove useless codes
Jiang-Jia-Jun pushed a commit that referenced this pull request Nov 6, 2025
* process transparent image

* english comments

* process transparency at downloading

* fix

* remove useless codes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants