-
-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
python311Packages.{onnxconverter-common,skl2onnx}: unpin protobuf #256603
Conversation
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/2696 |
Why this change? Has it been tested? If so, on what platform? Please use the form. |
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.
Those should be 2 commits and one should be a normal update commit
There are errors similar to the following when building these two packages.
It turns out that they both depend on onnx which depend on protobuf (not protobuf3). |
Here's the tricky thing. skl2onnx depends on onnx and onnxconverter-common. And onnxconverter-common also depends on onnx. |
If all parts can work with the latest protobuf, then that is fine. But please be clear about this in your pull request. The form exists for a reason. |
My bad. All done. |
Result of 12 packages built:
|
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)