-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
StateT Performance Regression #2220
Comments
Can't really get mad about a regression if it's for safety/correctness eh? Cool, hopefully |
We did actually know it was slightly slower, but we made the trade off. It would be very interesting to see if we could recover some of that. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Between 1.0.0 and 1.1.0 (and previously between 0.9 and 1.0.0),
StateT
has suffered a performance regression. I discovered this when updating versions for scalaz-and-cats.For:
we see this runtime drop from 8,920,953 ns to 9,744,808 ns. This is after a 2x slowdown between 0.9 and 1.0.0 which @iravid said had something to do with
IndexedStateT
. This 9.7 million ns number is around twice as slow as the equivalent ScalaZ, and 5 orders of magnitude slower than Haskell.The text was updated successfully, but these errors were encountered: