Skip to content

Conversation

@dh7
Copy link

@dh7 dh7 commented Jun 8, 2022

No description provided.

@vercel
Copy link

vercel bot commented Jun 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
clipdrop-api-sample-decompose-layers ✅ Ready (Inspect) Visit Preview Jun 22, 2022 at 7:21AM (UTC)
clipdrop-api-samples-text-remover ✅ Ready (Inspect) Visit Preview Jun 22, 2022 at 7:21AM (UTC)
remove-objects-tfjs ✅ Ready (Inspect) Visit Preview Jun 22, 2022 at 7:21AM (UTC)
sample-logo-removal ❌ Failed (Inspect) Jun 22, 2022 at 7:21AM (UTC)
sample-person-removal ❌ Failed (Inspect) Jun 22, 2022 at 7:21AM (UTC)
1 Ignored Deployment
Name Status Preview Updated
remove-background-bulk ⬜️ Ignored (Inspect) Jun 22, 2022 at 7:21AM (UTC)

Copy link
Contributor

@jblanche jblanche left a comment

Choose a reason for hiding this comment

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

Should we have a cleanup example too ?

"""call the API to remove the background"""
output_format = os.path.splitext(output_file)[1].lower()
img_in = open(input_file, 'rb')
files = {'image_file': ('image.jpeg', img_in, 'image/jpeg')}
Copy link
Contributor

Choose a reason for hiding this comment

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

What if the input image is a webp / png ?

Copy link
Author

Choose a reason for hiding this comment

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

it doesn't matter. I've tryed with png and it work (ligne 53). It seems that neither the API or the services cares

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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.

3 participants