FluxML / Zygote.jl

21st century AD
https://fluxml.ai/Zygote.jl/
Other
1.48k stars 210 forks source link

Mark AbstractFFT test broken #1452

Closed ToucheSir closed 1 year ago

ToucheSir commented 1 year ago

Having to look through every CI job to see if the failures are known is laborious. The Test stdlib gives us a way to automate this, so let's use it.

gaurav-arya commented 1 year ago

FYI that there isn't a clear timeline for when AbstractFFTs will get its next release, so if you'd like to fix the Zygote CI without waiting on it I'd recommend just merging this, we can always change it back later

ToucheSir commented 1 year ago

Thanks for the heads up. I think this PR is good to go then.