Skip to content

Conversation

davidmotson
Copy link
Contributor

@davidmotson davidmotson commented Jul 16, 2025

Adds Imagen Editing cases to the quickstart including:

  • Inpainting
  • Outpainting
  • Subject References
  • Style Transfer

@davidmotson davidmotson changed the title Davidmotson.imagen editing Add imagen editing cases to the Firebase AI quickstart Jul 16, 2025
@rlazo
Copy link
Contributor

rlazo commented Jul 23, 2025

Please add a description to the PR

@rlazo rlazo closed this Jul 23, 2025
@rlazo rlazo reopened this Jul 23, 2025
@davidmotson davidmotson requested a review from thatfiredev July 29, 2025 17:22
@rlazo
Copy link
Contributor

rlazo commented Aug 18, 2025

Can we make the alignment a dropdown and add a label so it's clear what the values mean?

The current setup isn't super intuitive. See screenshot

Screenshot 2025-08-18 at 6 03 11 PM

@rlazo
Copy link
Contributor

rlazo commented Aug 18, 2025

Also, probably "Select" is better? Additionally, can we make the buttons be side by side?

Screenshot 2025-08-18 at 6 05 42 PM

@rlazo
Copy link
Contributor

rlazo commented Aug 18, 2025

Style transfer is not a very clear example since we can't see the image of the cat used as style reference.

When running it, it was not clear what image it was expecting me to upload, and it was not clear what the reference style image was going to be, since I can't see it outside the code review

Screenshot 2025-08-18 at 6 13 10 PM

@davidmotson davidmotson requested a review from rlazo August 20, 2025 18:43
@davidmotson davidmotson requested a review from rlazo August 20, 2025 22:29
Copy link
Member

@thatfiredev thatfiredev left a comment

Choose a reason for hiding this comment

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

LGTM

@thatfiredev thatfiredev merged commit 9dffec2 into master Aug 26, 2025
7 checks passed
@davidmotson davidmotson deleted the davidmotson.imagen_editing branch August 31, 2025 10:14
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.

4 participants