[svistoi]

Adds pybind11 pinned version 3.0.4. 3.1 introduced:

• a bug: fix: do not run keep_alive for an overload that failed argument conversion pybind/pybind11#6154 (fixed in fix: never run keep_alive for a failed overload or failed return conversion pybind/pybind11#6183 (not yet released)).
• behavioral change: Expand float and complex strict mode to allow ints and ints/float (for PEP 484 compatibility). pybind/pybind11#5879

Which (respectively):

• Break frigate's usage of openvino when using intel GPU accelerated models.
• Is incompatible with python3Packages.scipp which explicitly requires 3.0.4 due to a different pybind11 3.1 issue.

Things done

I copied pybind11/default.nix and applied:

diff pkgs/development/python-modules/pybind11/3_0.nix pkgs/development/python-modules/pybind11/default.nix

33c33
<   version = "3.0.4";
---
>   version = "3.1.0";
40c40
<     hash = "sha256-QZKnIOopEDsiRFkc1qQ+DaDHoTNuEEgQVeiAL0sQqak=";
---
>     hash = "sha256-rzpe7CrgIa5Df2OrB/9mxIJd3X5DA7FX0C+w7TcmAoQ=";

• Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.
• Follows the automation/AI policy.
• Built on platform:
  • x86_64-linux
  • aarch64-linux
  • aarch64-darwin
• Tested, as applicable:
  • Basic functionality of all binary files, usually in ./result/bin/.
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Package tests at passthru.tests.
  • NixOS tests in nixos/tests.
  • Tests in lib/tests or pkgs/test for functions and "core" functionality.
• Nixpkgs Release Notes
  • Package update: when the change is major or breaking.
• NixOS Release Notes
  • Module addition: when adding a new NixOS module.
  • Module update: when the change is significant.

[svistoi]

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 578385
Commit: f25c54d2f276a8202d387c821681daa8b8d9cb51

---

x86_64-linux

✅ 2 packages built:

• python313Packages.pybind11_3_0
• python314Packages.pybind11_3_0

[doronbehar]

I'd put the openvino and scipp migrations here as part of this PR, to make it clear why is this new 3.0.4 derivation is needed, and to make it easier for everyone to test whether this works.

[doronbehar]

Note also my changes to the PR's description.

[svistoi]

I'd put the openvino and scipp migrations here as part of this PR, to make it clear why is this new 3.0.4 derivation is needed, and to make it easier for everyone to test whether this works.

The other PR request for change asked for pybind package to be a separate merge. I tested openvino with downgrade
#576186

[mweinelt]

Nah, it asked for it to be a separate commit. It could have very well gone in to the same PR.