hmm, I know this isn't your code, but I'm not a fan of this: we are still performing an equality comparison between floats. Can we use `assert_all_close(..., atol=1e-4)` instead?