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

Error when using @tf.function in release 0.17.0 #426

Closed
siebertdarius opened this issue May 2, 2024 · 1 comment
Closed

Error when using @tf.function in release 0.17.0 #426

siebertdarius opened this issue May 2, 2024 · 1 comment

Comments

@siebertdarius
Copy link

Hello Nvidia,

I have an issue using the @tf.function in the new sionna release. The error (see below) occured and can be reproduced when running the notebook code for learned materials from your diff-rt-calibration project with sionna 0.17.0 release.

This code works perfectly for sionna 0.16.2. Also, the error refers to the doppler shifts which are touched in release 0.17.0, so I suspect an issue within this new release.

When @tf.function is deleted in the code, no error occurs but the results differ drastically from the usual results.

    TypeError: in user code:

    File "/tmp/ipykernel_481/3933607846.py", line 61, in training_step  *
        paths = scene.compute_fields(*traced_paths,
    File "/opt/conda/lib/python3.11/site-packages/sionna/rt/scene.py", line 620, in compute_fields  *
        output = self._solver_paths.compute_fields(spec_paths, diff_paths,
    File "/opt/conda/lib/python3.11/site-packages/sionna/rt/solver_paths.py", line 754, in compute_fields  *
        spec_paths.doppler = self._compute_doppler_shifts(spec_paths,
    File "/opt/conda/lib/python3.11/site-packages/sionna/rt/solver_paths.py", line 4127, in _compute_doppler_shifts  *
        doppler = tf.where(objects_mask, 0, doppler)

    TypeError: Input 'e' of 'SelectV2' Op has type float32 that does not match type int32 of argument 't'.
Jeija added a commit to Jeija/sionna that referenced this issue May 10, 2024
Jeija added a commit to Jeija/sionna that referenced this issue May 10, 2024
Signed-off-by: Florian Euchner <florian.euchner@gmail.com>
@SebastianCa
Copy link
Collaborator

Hi @siebertdarius,

thank you for reporting this issue. The issue will be fixed in the next release of Sionna (see #436).

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