jrmccluskey commented on PR #34700: URL: https://github.com/apache/beam/pull/34700#issuecomment-2839155916
You should be able to ignore prism and yaml failures, those are generally flaky and not impacted by anything here. The EmbeddingsManager class is effectively a composite PTransform that produces a RunInference transform with a more traditional model handler, I cannot see a reason why that wouldn't work with the remote handler implementation. Not having a model for image embeddings is fine since you've clearly labeled the class as a text embedding model, we can always add images / multimodal implementations later as APIs become available. -- 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
