-
Notifications
You must be signed in to change notification settings - Fork 474
GCP Vertex AI LLM Integration #11225
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Based on the screenshot attached to the issue, here below are some of the suggestions for improvements
|
I see that the screenshot attached to the PR is not updated. So, some of the comments mentioned above may be obsolete and may not need any further action. |
|
@@ -0,0 +1,74 @@ | |||
--- | |||
description: Pipeline for parsing GCP Vertex AI metrics. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have used available semconv metric names for AWS bedrock field names.
Is the below field name pattern the preferred pattern for GCP?
Is there any reason why the TSDS enablement is not present? |
We are doing this for Tech Preview and to start getting the initial feedbacks. |
💚 Build Succeeded
History
cc @ishleenk17 |
|
Package gcp_vertexai - 0.0.1 containing this change is available at https://epr.elastic.co/search?package=gcp_vertexai |
* Update null checks anf if checks in the rename * Update changelog.yml * Inital PR for GCP Vertex AI Integration * update dashboard * Update dashboard * Add metrics and update dashboard * add logo image * update PR id * Add fields, update dashboard * update dashboard * update title * Update dashboard * draft documentation * update dashboard screenshot * update metric type for fields * documentation update * update README * update unit type * update dashboard * update dashboard image * merge bytes transferred in a single chart --------- Co-authored-by: muthu-mps <muthukumar.paramasivam@elastic.co>
* Update null checks anf if checks in the rename * Update changelog.yml * Inital PR for GCP Vertex AI Integration * update dashboard * Update dashboard * Add metrics and update dashboard * add logo image * update PR id * Add fields, update dashboard * update dashboard * update title * Update dashboard * draft documentation * update dashboard screenshot * update metric type for fields * documentation update * update README * update unit type * update dashboard * update dashboard image * merge bytes transferred in a single chart --------- Co-authored-by: muthu-mps <muthukumar.paramasivam@elastic.co>
GCP Vertex AI LLM Integration
Checklist
changelog.yml
file.Author's Checklist
Related issues
Screenshots