coiled / benchmarks

BSD 3-Clause "New" or "Revised" License
28 stars 17 forks source link

Fix numerical instability in DuckDB Q15 #1454

Closed hendrikmakait closed 6 months ago

hendrikmakait commented 6 months ago

The revenue calculation was numerically unstable and Q15 could (incorrectly) return no results.

hendrikmakait commented 6 months ago

FWIW, I guess that the other libraries have a problem with this as well. @phofl: This kind of instability is something that Dask will become prone to once we disable CSE. If you have a good intuition on how to detect this on query generation, I'd love to flag/handle this.