aIbrahiim commented on code in PR #38090:
URL: https://github.com/apache/beam/pull/38090#discussion_r3054317819


##########
sdks/python/setup.py:
##########
@@ -185,7 +185,7 @@ def cythonize(*args, **kwargs):
 ]
 
 ml_adk_dependency = [
-    'google-adk==1.17.0',
+    'google-adk==1.28.1',
     'opentelemetry-api==1.37.0',
     'opentelemetry-sdk==1.37.0',
     'opentelemetry-exporter-otlp-proto-http==1.37.0',

Review Comment:
   I had to pin opentelemetry-exporter-otlp-proto-http because otherwise pip 
kept grabbing a newer 1.40.x build that wants opentelemetry-sdk~=1.40, while 
ADK only lines up with 1.37.x so resolution blew up unless i locked the 
exporter to the same minor as the SDK @damccorm 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to