-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
vertex-ai
: add google_vertex_ai_model
resource
#12490
base: main
Are you sure you want to change the base?
Conversation
…opy (#12074) Co-authored-by: Sarah French <[email protected]> Co-authored-by: Sarah French <[email protected]>
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_vertex_ai_model" "primary" {
artifact_uri = # value needed
container_spec {
args = # value needed
command = # value needed
deployment_timeout = # value needed
env {
name = # value needed
value = # value needed
}
grpc_ports {
container_port = # value needed
}
health_probe {
exec {
command = # value needed
}
period_seconds = # value needed
timeout_seconds = # value needed
}
health_route = # value needed
image_uri = # value needed
ports {
container_port = # value needed
}
predict_route = # value needed
shared_memory_size_mb = # value needed
startup_probe {
exec {
command = # value needed
}
period_seconds = # value needed
timeout_seconds = # value needed
}
}
encryption_spec {
kms_key_name = # value needed
}
metadata {
config {
algorithm_config {
tree_ah_config {
leaf_node_embedding_count = # value needed
leaf_nodes_to_search_percent = # value needed
}
}
approximate_neighbors_count = # value needed
dimensions = # value needed
distance_measure_type = # value needed
feature_norm_type = # value needed
shard_size = # value needed
}
}
metadata_schema_uri = # value needed
pipeline_job = # value needed
predict_schemata {
instance_schema_uri = # value needed
parameters_schema_uri = # value needed
prediction_schema_uri = # value needed
}
supported_export_formats {
exportable_content = # value needed
}
version_aliases = # value needed
}
|
Tests analyticsTotal tests: 61 Click here to see the affected service packages
Action takenFound 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
Broken up into separate PRs.
:COPY
vertex-ai
: addgoogle_vertex_ai_models
resource with create-via-copy #12074:UPLOAD
vertex-ai
: add:UPLOAD
support for model resource #12075 - The merging of this PR will introduce renaming ofdisplayName
->modelName
. This is meant to make it easier for users to differentiate how to COPY or UPLOAD a model (COPY requiressource_model
field while UPLOAD requiresdisplayName
field but will be renamed to bemodelName
.explanation_spec_examples
vertex-ai
: add:UPLOAD
explanation_spec_examples
tests #12170explanation_gradients
vertex-ai
: add:UPLOAD
integrated_gradients
tests #12171explanation_xrai
vertex-ai
: add:UPLOAD
xrai_attributions
test #12173explanation_shapley
vertex-ai
: add:UPLOAD
sampled_shapley_attributions
test #12172Once each individual checkbox is addressed we can move this out of draft.
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.