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
python3Packages.arviz: init at 0.5.1 #73988
Conversation
54c5188
to
f9a117b
Compare
|
f9a117b
to
d764d8d
Compare
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.
commit message should be:
python3Packages.arviz: init at 0.5.1
arviz alone would mean you are putting a top level attribute in all-packages.nix
d764d8d
to
97f7f0a
Compare
This should be fixed. I was able to fix the first remotely from my phone earlier today, but I didn't have access to Github to update the title. |
@veprbl Can you give output? I have a MacOSX machine I can test on. But the issue might be (and commonly is based on the other tests) other packages in the ecosystem being just plain broken on NixOS. I can disable that test if we cannot come to a satisfactory method of making it work. However, it should be noted that when @risicle left that review I had tests disabled. It was only at the prompting of @jonringer that I added them back in. |
97f7f0a
to
ec1e2ac
Compare
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.
Builds and works on darwin
@GrahamcOfBorg build python37Packages.arviz |
@GrahamcOfBorg build python37Packages.arviz |
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 would still like to see this squashed, otherwise LGTM
Motivation for this change
Introduce the python package arviz, which is a module for helping visualize Bayesian distributions. Particularly useful in Bayesian Neural Network analysis.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)