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.pyro-ppl: remove due to prolonged breakage #95350
Conversation
homepage = "http://pyro.ai"; | ||
license = lib.licenses.mit; | ||
maintainers = with lib.maintainers; [ teh ]; | ||
broken = true; |
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.
This thing looks extremely actively maintained (last commit 1 hour ago). @stites @drewrisinger either of you used it? Maybe it was just broken due to an old pytorch, and now just needs an uprev.
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.
could be that it has a very large and fragile dependency graph, in which case, it would be hard to maintain in context on the nixpkgs python set
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'm using actively but we're not on master nixpkgs. Might indeed be easier to maintain out of core nixpkgs.
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.
@FRidh might have an opinion because he updated to the recent version.
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.
looks like it was part of a large update, and pyro-ppl is "just a leaf" package, so it wasn't looked at for regressions
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.
@bhipple never used it.
Is this still necessary? The package builds and works fine on current master. |
Motivation for this change
Was marked broken in oct 2019
#95265
cc @teh
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)