aIbrahiim commented on code in PR #38090:
URL: https://github.com/apache/beam/pull/38090#discussion_r3053214480
##########
sdks/python/setup.py:
##########
@@ -546,21 +560,22 @@ def get_portability_package_data():
'datatable',
# tensorflow-transform requires dill, but doesn't set dill as a
# hard requirement in setup.py.
- 'dill', # match tft extra.
+ 'dill',
+ # match tft extra.
'tensorflow_transform>=1.14.0,<1.15.0',
# TFT->TFX-BSL require pandas 1.x, which is not compatible
# with numpy 2.x
Review Comment:
if you mean protobuf<4, i dropped that cap because with ADK/OTel in the
same extra it fought protobuf>=5. ml_test no longer pulls ADK, so that specific
cap isnt what unblocks resolution anymore, we can revisit a narrower cap
separately if [gcp]+[ml_test] still allows it.
--
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]
