Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: add image compression for DALL-E generated images #10408

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

fix: add image compression for DALL-E generated images #10408

wants to merge 4 commits into from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Dec 19, 2024 •
edited by dillchen
Loading

Link to Issue

Closes: #10244

Description of Changes

  • Added image compression utility using sharp library
  • Updated DALL-E configuration to use 'dall-e-3' model and 1024x1024 size
  • Added compression step before uploading images to blob storage
  • Ensures images stay under 500kb size limit
  • Added @types/sharp for TypeScript support

Test Plan

  • Verify Image generation uses Dalle-3 in token creation, community creation, and contest creation
  • Verify that users can save and complete token, community, and contest creation with compressed images

Link to Devin run: https://app.devin.ai/sessions/7445ab1d09a14906b6fd1a8a7fef96e7

Deployment Plan

N/A

Other Considerations

@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Add "(aside)" to your comment to have me ignore it.

@devin-ai-integration devin-ai-integration bot force-pushed the devin/1734649172-add-image-compression branch from a90b8c2 to b3bc57c Compare December 19, 2024 23:21
@dillchen dillchen closed this Dec 20, 2024
@dillchen dillchen reopened this Dec 20, 2024
@dillchen dillchen requested review from timolegros and removed request for dillchen December 20, 2024 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilijabojanovic ilijabojanovic Awaiting requested review from ilijabojanovic

@timolegros timolegros Awaiting requested review from timolegros

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dillchen