Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

to_tensorflow_function() dependency #161

Open
unsolicitedcrap opened this issue Jan 17, 2025 · 3 comments
Open

to_tensorflow_function() dependency #161

unsolicitedcrap opened this issue Jan 17, 2025 · 3 comments
Labels
bug Something isn't working

Comments

@unsolicitedcrap
Copy link

Using the python api, the YDF method to_tensorflow_function() defined in .../python3.12/dist-packages/ydf/model/generic_model.py:1714 calls _get_export_tf() which depends on tensorflow_decision_forests. While YDF supports tensorflow 2.18.0 with python 3.12, tensorflow_decision_forests has no python 3.12 build.

@rstz rstz added the bug Something isn't working label Jan 20, 2025
@rstz
Copy link
Collaborator

rstz commented Jan 20, 2025

Thank you for notifying us. At this time, a fix would require a release of TF-DF with Python 3.12, since the TF dependency is needed to create a TF model.

@unsolicitedcrap
Copy link
Author

As a maintainer of ydf and tfdf, do you have a rough timeframe when this might be addressed? I'm running into issues with the build process when I attempt it myself and I'm not well versed in bazel.

@rstz
Copy link
Collaborator

rstz commented Jan 31, 2025

I've learned never to give timeframes BUT I plan on doing it with the next release of Tensorflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants