-
Notifications
You must be signed in to change notification settings - Fork 62
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
Showing that random-fu is x4 faster than mwc and thus monad-bayes #321
Conversation
idontgetoutmuch
commented
Oct 13, 2023
We may be comparing apples and oranges here and one thing I plan to try is replacing |
I'd also like to benchmark sampling in Haskell against other languages. But is a PR here the best place to do these experiments? |
Yes, I guess it is, but in a separate PR (and a separate benchmarking suite). |
# Make sure you have direnv >= 2.30 | ||
use flake --extra-experimental-features nix-command --extra-experimental-features flakes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is that change intentional?
( | ||
import | ||
( | ||
let | ||
lock = builtins.fromJSON (builtins.readFile ./flake.lock); | ||
in | ||
fetchTarball { | ||
url = "https://github.com/edolstra/flake-compat/archive/${lock.nodes.flake-compat.locked.rev}.tar.gz"; | ||
sha256 = lock.nodes.flake-compat.locked.narHash; | ||
} | ||
) | ||
{src = ./.;} | ||
) | ||
.defaultNix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this was also not intentional?
@turion I just tried
but have type errors (if I uncomment the definition for EDIT: But this seems to work (at least it type checks):
|
Replaced by #323 |