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

feat(vertexai): add google_vertex_ai_index for Vertex AI Matching Engine #13132

Commits on Nov 29, 2022

  1. feat(vertexai): add google_vertex_ai_index for Vertex AI Matching Eng…

    …ine (hashicorp#6728)
    
    * feat: add google_vertex_ai_index for Vertex AI Matching Engine
    
    * fix: increase timeouts to 60 min because 20 wasn't enough for creation
    
    * fix: change coe to make name computed instead of an input
    
    * fix: use costom flatten code to ignore_read a nested property's field
    
    * fix: add skip_import_test: true to the auto-gen test
    
    * feat: add a test with a manually updated ImportStateVerifyIgnore
    
    * Apply suggestions from code review [ci skip]
    
    Update descriptions based on the suggestions
    
    Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
    
    * refactor: use ignore_read_extra instead of a manual test
    
    * fix: use an empty object for bruteForceConfig
    
    * feat: define additional fields to api.yaml
    
    * feat: add an example to increase test coverage
    
    * feat: deal with contentsDeltaUri as an updatable field
    
    * fix: fix the error because the cosine distance type only supports unit l2 norm type
    
    This is the error message from the endpoint:
    "Index with `COSINE_DISTANCE` distanceMeasureType currently only supports `UNIT_L2_NORM` featureNormType."
    
    * feat: remove approximate_neighbors_count from an example with brute_force_config
    
    approximate_neighbors_count is required if tree-AH algorithm is used.
    from https://cloud.google.com/vertex-ai/docs/matching-engine/configuring-indexes#brute-force-config
    
    * test: add a handwritten test for patch
    
    * fix: add update_mask: true to use the mask as a url param
    
    * refactor: put 'input: true' on the fields patch couldn't update
    
    * feat: use custom pre update code for a nested object
    
    * fix: update the handwritten test accordingly
    
    * feat: add custom flatten code for is_complete_overwrite
    
    Co-authored-by: Stephen Lewis (Burrows) <[email protected]>
    Signed-off-by: Modular Magician <[email protected]>
    modular-magician and melinath committed Nov 29, 2022
    Configuration menu
    Copy the full SHA
    48f83e2 View commit details
    Browse the repository at this point in the history