Skip to content

Commit 78a3889

Browse files
authored
Merge pull request #37735 from apache/cp-37721
Cherrypick #37721 to release-2.72
2 parents 3d11a05 + 5d2aa20 commit 78a3889

2 files changed

Lines changed: 26 additions & 12 deletions

File tree

buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3165,14 +3165,16 @@ class BeamModulePlugin implements Plugin<Project> {
31653165
def distTarBall = "${pythonRootDir}/build/apache-beam.tar.gz"
31663166
def packages = "gcp,test,aws,azure,dataframe"
31673167
def extra = project.findProperty('beamPythonExtra')
3168-
if (extra) {
3169-
packages += ",${extra}"
3170-
}
3171-
31723168
project.exec {
31733169
executable 'sh'
31743170
args '-c', ". ${project.ext.envdir}/bin/activate && pip install --pre --retries 10 ${distTarBall}[${packages}]"
31753171
}
3172+
if (extra) {
3173+
project.exec {
3174+
executable 'sh'
3175+
args '-c', ". ${project.ext.envdir}/bin/activate && pip install --pre --retries 10 ${distTarBall}[${extra}]"
3176+
}
3177+
}
31763178
}
31773179
}
31783180

sdks/python/setup.py

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,15 @@ def cythonize(*args, **kwargs):
163163
milvus_dependency = ['pymilvus>=2.5.10,<3.0.0']
164164

165165
ml_base = [
166-
'embeddings',
166+
'embeddings>=0.0.4', # 0.0.3 crashes setuptools
167167
'onnxruntime',
168168
'langchain',
169169
'sentence-transformers>=2.2.2',
170170
'skl2onnx',
171-
'pillow',
172-
'pyod',
171+
'pyod>=0.7.6', # 0.7.5 crashes setuptools
173172
'tensorflow',
173+
# tensorflow transient dep, lower versions not compatible with Python3.10+
174+
'absl-py>=0.12.0',
174175
'tensorflow-hub',
175176
'tf2onnx',
176177
'torch',
@@ -391,6 +392,7 @@ def get_portability_package_data():
391392
'numpy>=1.14.3,<2.5.0', # Update pyproject.toml as well.
392393
'objsize>=0.6.1,<0.8.0',
393394
'packaging>=22.0',
395+
'pillow>=12.1.1,<13',
394396
'pymongo>=3.8.0,<5.0.0',
395397
'proto-plus>=1.7.1,<2',
396398
# 1. Use a tighter upper bound in protobuf dependency to make sure
@@ -467,7 +469,7 @@ def get_portability_package_data():
467469
'pg8000>=1.31.5',
468470
"PyMySQL>=1.1.0",
469471
'oracledb>=3.1.1'
470-
] + milvus_dependency,
472+
],
471473
'gcp': [
472474
'cachetools>=3.1.0,<7',
473475
'google-api-core>=2.0.0,<3',
@@ -551,7 +553,9 @@ def get_portability_package_data():
551553
'p312_ml_test': [
552554
'datatable',
553555
] + ml_base,
554-
'p313_ml_test': ml_base,
556+
# maintainer: milvus tests only run with this extension. Make sure it
557+
# is covered by docker-in-docker test when changing py version
558+
'p313_ml_test': ml_base + milvus_dependency,
555559
'aws': ['boto3>=1.9,<2'],
556560
'azure': [
557561
'azure-storage-blob>=12.3.2,<13',
@@ -584,7 +588,11 @@ def get_portability_package_data():
584588
# For more info, see
585589
# https://docs.google.com/document/d/1c84Gc-cZRCfrU8f7kWGsNR2o8oSRjCM-dGHO9KvPWPw/edit?usp=sharing
586590
'torch': ['torch>=1.9.0,<2.8.0'],
587-
'tensorflow': ['tensorflow>=2.12rc1,<2.21'],
591+
'tensorflow': [
592+
'tensorflow>=2.12rc1,<2.21',
593+
# tensorflow transitive dep
594+
'absl-py>=0.12.0'
595+
],
588596
'transformers': [
589597
'transformers>=4.28.0,<4.56.0',
590598
'tensorflow>=2.12.0',
@@ -593,7 +601,9 @@ def get_portability_package_data():
593601
'ml_cpu': [
594602
'tensorflow>=2.12.0',
595603
'torch==2.8.0+cpu',
596-
'transformers>=4.28.0,<4.56.0'
604+
'transformers>=4.28.0,<4.56.0',
605+
# tensorflow transient dep
606+
'absl-py>=0.12.0'
597607
],
598608
'redis': ['redis>=5.0.0,<6'],
599609
'tft': [
@@ -610,7 +620,9 @@ def get_portability_package_data():
610620
'tensorflow==2.11.0',
611621
'tf2onnx==1.13.0',
612622
'skl2onnx==1.13',
613-
'transformers==4.25.1'
623+
'transformers==4.25.1',
624+
# tensorflow transient dep
625+
'absl-py>=0.12.0'
614626
],
615627
'xgboost': ['xgboost>=1.6.0,<2.1.3', 'datatable==1.0.0'],
616628
'tensorflow-hub': ['tensorflow-hub>=0.14.0,<0.16.0'],

0 commit comments

Comments
 (0)