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.

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

GCP Pipeline Jobs #315

Merged
merged 9 commits into from
Aug 30, 2023
Merged

GCP Pipeline Jobs #315

merged 9 commits into from
Aug 30, 2023

Conversation

fedefernandez
Copy link
Contributor

API to expose the VertexAI endpoints for custom models (fine-tuning)

@fedefernandez fedefernandez changed the title GCP Pipeline Jobs [DRAFT] GCP Pipeline Jobs Aug 11, 2023
Copy link
Contributor

@raulraja raulraja left a comment

Choose a reason for hiding this comment

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

Looking great @fedefernandez !!

Copy link
Contributor

@nomisRev nomisRev left a comment

Choose a reason for hiding this comment

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

This PR looks pretty good to me so far. Only added a small reply to Raul's comment.

@Intex32
Copy link
Member

Intex32 commented Aug 28, 2023

This PR looks pretty good to me so far. Only added a small reply to Raul's comment.

I talked to the others, Fede is currently on vacation. You can apply your suggestions and finish this pr, as far as I am concerned.

@Intex32 Intex32 self-assigned this Aug 29, 2023
@Intex32
Copy link
Member

Intex32 commented Aug 29, 2023

I made some changes considering previous comments and at my own discretion. I kindly request to approve soon as I then can carry on with further reworking the GCP code.

@Intex32 Intex32 marked this pull request as ready for review August 30, 2023 07:21
nomisRev
nomisRev previously approved these changes Aug 30, 2023
Copy link
Contributor

@nomisRev nomisRev left a comment

Choose a reason for hiding this comment

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

Your changes look good to me @Intex32! Only need to fix the merge conflicts, and I think we're ready to merge 🙌

@Intex32
Copy link
Member

Intex32 commented Aug 30, 2023

I'm currently on it. @nomisRev

@Intex32 Intex32 changed the title [DRAFT] GCP Pipeline Jobs GCP Pipeline Jobs Aug 30, 2023
# Conflicts:
#	examples/kotlin/src/main/kotlin/com/xebia/functional/xef/conversation/gpc/Chat.kt
#	integrations/gcp/src/commonMain/kotlin/com/xebia/functional/xef/gcp/GcpChat.kt
#	integrations/gcp/src/commonMain/kotlin/com/xebia/functional/xef/gcp/GcpClient.kt
Copy link
Contributor

@javipacheco javipacheco left a comment

Choose a reason for hiding this comment

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

🚀 LGTM!

@Intex32 Intex32 merged commit 5e84a22 into main Aug 30, 2023
@Intex32 Intex32 deleted the feature/pipeline-jobs branch August 30, 2023 13:28
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.

5 participants