Skip to content

Adding Vertex AI Imagen models support #2144

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.

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Marzouki-Sami
Copy link

Closes gh-2133

Signed-off-by: Marzouki-Sami [email protected]

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

@Marzouki-Sami Marzouki-Sami marked this pull request as draft January 29, 2025 07:07
@Marzouki-Sami Marzouki-Sami marked this pull request as ready for review January 29, 2025 07:12
@markpollack
Copy link
Member

@ddobrin can you please take a look?

@turchinc
Copy link
Contributor

Any update here? I am also interested in this feature.

@byungjunn
Copy link

I am currently using this feature as a custom-made version. I hope it gets updated soon. If help is needed, I would be willing to participate.

@ilayaperumalg
Copy link
Member

@byungjunn Thanks! Any help reviewing the PR is much appreciated.

@byungjunn
Copy link

@Marzouki-Sami
https://spring.io/blog/2025/03/25/spring-ai-update-to-snapshots
After the update, there have been many changes, so I think you need to rebase onto the main branch.

Marzouki-Sami and others added 5 commits June 16, 2025 11:09
Closes spring-projectsgh-2133

Signed-off-by: Marzouki-Sami [email protected]
Signed-off-by: Marzouki-Sami <[email protected]>
…gen/pom.xml


Updating the version.

Co-authored-by: ByungJun <[email protected]>
Signed-off-by: Sami Marzouki <[email protected]>
…gen/pom.xml


changing artifactId

Co-authored-by: ByungJun <[email protected]>
Signed-off-by: Sami Marzouki <[email protected]>
…gen/pom.xml


removing unnecessary spaces

Co-authored-by: ByungJun <[email protected]>
Signed-off-by: Sami Marzouki <[email protected]>
Co-authored-by: ByungJun <[email protected]>
Signed-off-by: Sami Marzouki <[email protected]>
@Marzouki-Sami
Copy link
Author

@byungjunn @ilayaperumalg @markpollack i had some issues with resolving the conflict and to resolve the conflict i had to remove the commits, doing that closed this PR is it possible to reopen it, if not i will create a new PR and link it to this one.

@ilayaperumalg ilayaperumalg reopened this Jun 17, 2025
@Marzouki-Sami Marzouki-Sami force-pushed the gh-2133 branch 2 times, most recently from 7fef979 to 54f3ba7 Compare June 17, 2025 09:47
sami.marzouki and others added 2 commits June 17, 2025 10:49
Signed-off-by: Marzouki-Sami <[email protected]>
@Marzouki-Sami
Copy link
Author

@byungjunn thanks for the review the mentioned points above are fixed. Can you check them?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: adding Vertex Ai Imagen models support
5 participants