This repository has been archived by the owner on Sep 28, 2024. It is now read-only.

Consider using fft to rfft #151

Open

dynamic-queries opened this issue Nov 12, 2023 · 1 comment

dynamic-queries commented Nov 12, 2023

I ran into this issue a few months ago. So apologies for the late report.

For the FNO, rfft is used.
From FluxML/Zygote.jl#1406 , see that Zygote's gradient is wrong.
A quick fix is to change this to fft for now.

Member

ChrisRackauckas commented Nov 12, 2023

That seems like a fine temporary workaround. I'd accept it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.