google/ml-metadata

Python 3.9 support

chensun opened this issue · 9 comments

Setup.py requirements indicates it should support Python 3.9

python_requires='>=3.7,<4',

However, there's no wheel for 3.9 released:
https://pypi.org/project/ml-metadata/1.5.0/#files

I took a quick look at the dependency packages, all of them seems to support Python 3.9. Is there something blocking ml-metadata from supporting Python 3.9, or it's just a miss in release configuration?

Hi Chen Sun,

Yes, MLMD supports Python 3.9. Let me check with our release team on this. Thanks for bringing this to our notice!

After checking with our release team, MLMD will not proceed with Python 3.9 support until Apache Beam support Python 3.9, and it seems that they have not support it yet: https://pypi.org/project/apache-beam/2.35.0/#files.

Let's keep this issue open and resolve it after MLMD officially support Python 3.9.

Thanks for the reply @BrianSong.

I'm not sure why apache-beam is in the way. It's a not a dependency per se:

ml-metadata/setup.py

Lines 150 to 156 in c28a77f

install_requires=[
'absl-py>=0.9,<0.13',
'attrs>=20.3,<21',
'grpcio>=1.8.6,<2',
'protobuf>=3.13,<4',
'six>=1.10,<2',
],

As internally, MLMD will be released together with TFX-related teams. As some of the TFX-related teams depend on apache beam, our release team decided to support python 3.9 after apache beam to avoid confusion.

Hey @BrianSong 👋 Apache Beam now supports Python 3.9 2.37.0, is it possible to update the MLMD to also support? Let me know if I can help in anyway. Thanks!

Hi @alexjajones, thanks for letting us know. Let me talk to the internal release team.

From the internal release team: will be supporting py39 for TFX libs from next release train i.e
TFX 1.8.0/0.39.0.

@BrianSong I see that ml-metadata was updated to support python 3.9.

However, it doesn't look like pybind11 was updated. ml-metadata is currently using 2.4.3 but python 3.9 support wasn't fully supported until 2.6.0

@CandiedCode MLMD now supports Python 3.10 and pybind11 2.10.