Rebased on master, no structural changes.

The CLI apparently expects the onnxruntime package. Maybe we can add testers.testVersion to run the CLI as a further test.

$ ./result/bin/insightface-cli
Traceback (most recent call last):
  File "/nix/store/l2j9xhb2zbl21id24xy5kgly0vb7mx74-python3.11-insightface-0.7.3/lib/python3.11/site-packages/insightface/__init__.py", line 8, in <module>
    import onnxruntime
ModuleNotFoundError: No module named 'onnxruntime'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/nix/store/l2j9xhb2zbl21id24xy5kgly0vb7mx74-python3.11-insightface-0.7.3/bin/.insightface-cli-wrapped", line 6, in <module>
    from insightface.commands.insightface_cli import main
  File "/nix/store/l2j9xhb2zbl21id24xy5kgly0vb7mx74-python3.11-insightface-0.7.3/lib/python3.11/site-packages/insightface/__init__.py", line 10, in <module>
    raise ImportError(
ImportError: Unable to import dependency onnxruntime. 

Added small compatibility fix for mxnet (to support numpy >= 1.24), and addressed review comments.
The mxnet patch is not made against upstream and fetchpatched because the newest upstream version has already evolved to another incompatible version, so that patch wouldn't apply, and mxnet seems to be unmaintained.

The CLI apparently expects the onnxruntime package. Maybe we can add testers.testVersion to run the CLI as a further test.

Good idea. I've now added this test but since there is no "--version" for insightface, I've set version = "help" to verify at least the help output.

(Fixed accidentally deleted character 😅, package builds and runs now)

mxnet fails to build on darwin.

error: unknown warning option '-Werror=maybe-uninitialized'; did you mean '-Werror=uninitialized'? [-Werror,-Wunknown-warning-option]

Can you try the following? Copied from pkgs/tools/networking/tgt/default.nix.

env.NIX_CFLAGS_COMPILE = toString [
	# Needed with GCC 12
	"-Wno-error=maybe-uninitialized"
];

Well that was dumb. I really need to start reading the actual error message.

The necessary change is

diff --git a/pkgs/applications/science/math/mxnet/default.nix b/pkgs/applications/science/math/mxnet/default.nix
index 1e7af371c50..d65de87d8eb 100644
--- a/pkgs/applications/science/math/mxnet/default.nix
+++ b/pkgs/applications/science/math/mxnet/default.nix
@@ -56,7 +56,6 @@ stdenv.mkDerivation rec {
 
   env.NIX_CFLAGS_COMPILE = toString [
     # Needed with GCC 12
-    "-Wno-error=maybe-uninitialized"
     "-Wno-error=uninitialized"
   ];

The necessary change is [...]

Adjusted, I just can't test darwin myself unfortunately :D

The change is needed on the native library, not the python library, see the diff. 😆

Guess I should also start to read comments properly 😅. Added as suggested now.

@ofborg build python310Packages.mxnet

Builds for me on both darwin systems.