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

Compatibility between Tensorflow 2.10-2.12 and Sionna 0.16.2 #376

Closed
GKasperF opened this issue Mar 26, 2024 · 2 comments
Closed

Compatibility between Tensorflow 2.10-2.12 and Sionna 0.16.2 #376

GKasperF opened this issue Mar 26, 2024 · 2 comments

Comments

@GKasperF
Copy link

This is a rather minor issue, but it seems some changes in 0.16.2 made it not entirely compatible with previous TensorFlow versions.
For example, the method sample_positions from the class CoverageMap has the following line:

reps = tf.math.ceil(tf.math.divide_no_nan(tf.cast(batch_size, tf.int32),

However, up to TensorFlow 2.12, divide_no_nan only takes inputs with dtype float32 or float64. This was updated in 2.13.

Please clarify in the documentation the TensorFlow requirements.

@SebastianCa
Copy link
Collaborator

Hi @GKasperF,

thank you for reporting this issue and its detailed investigation. We will fix the issue with the next release of Sionna.
An obvious workaround is to use TensorFlow 2.13 (and newer).

@SebastianCa
Copy link
Collaborator

Fixed in Sionna v0.17

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