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

Keras 3 compatibility #573

Open
hafezmg48 opened this issue Sep 13, 2024 · 2 comments
Open

Keras 3 compatibility #573

hafezmg48 opened this issue Sep 13, 2024 · 2 comments

Comments

@hafezmg48
Copy link

Hi, Thanks for your great contribution.

Sionna current version 0.18.0 is not implemented on Keras 3. However, keras 3 has many important features and it would be great to have sionna in keras 3 so that it can be integrated in models that are developed with Keras 3. Is this possible?
Thanks

@jhoydis
Copy link
Collaborator

jhoydis commented Sep 16, 2024

Hi @hafezmg48,

There reason why Keras 3 is not supported is that it does not work with complex-valued Tensors. Once this issue is resolved (keras-team/keras#19872), Sionna will be compatible with Keras 3.

However, note that Sionna will only support the TF backend.

@hafezmg48
Copy link
Author

hafezmg48 commented Sep 23, 2024

Hi @hafezmg48,

There reason why Keras 3 is not supported is that it does not work with complex-valued Tensors. Once this issue is resolved (keras-team/keras#19872), Sionna will be compatible with Keras 3.

However, note that Sionna will only support the TF backend.

@jhoydis Thanks for considering this issue. Although we have discussed this and you correctly mentioned that the reason is that Keras 3 is not supporting complex tensors, I would like to also mention another related problem for your future reference. That is, the current supported TF version 2.15 is only compatible with at most the CUDA 11 which is currently outdated for many other AI projects which makes the integration of Sionna to those very hard (if we were to use the gpu for sionna). I believe this issue also has the same root as what you mentioned that TF =>2.16 works with keras 3 and keras 3 is not supporting complex-tensor, but wanted to specifically mention it for the reference. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants