jrmccluskey commented on code in PR #31514:
URL: https://github.com/apache/beam/pull/31514#discussion_r1628065750


##########
sdks/python/apache_beam/ml/transforms/embeddings/vertex_ai_test.py:
##########
@@ -245,5 +248,45 @@ def test_mltransform_to_ptransform_with_vertex(self):
           ptransform_list[i]._model_handler._underlying.model_name, model_name)
 
 
+@unittest.skipIf(
+    VertexAIImageEmbeddings is None, 'Vertex AI Python SDK is not installed.')
+class VertexAIImageEmbeddingsTest(unittest.TestCase):

Review Comment:
   I am open to changing this to an integration test since we're accessing a 
GCS bucket; however, we aren't actually loading the image on-machine and that 
load is happening on the vertex side



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to