| 23 Oct 2024 |
connor (burnt/out) (UTC-8) | Okay I looked at it and have no idea 🤷‍♂️ | 16:32:07 |
connor (burnt/out) (UTC-8) | The whole ONNX ecosystem is difficult to package for Nix because they all use both git submodules AND CMake’s fetchcontent functionality, making it super difficult to package with stuff we already provide. For some packages, they build with flags we don’t, or have patches they apply before building, so it’s painful. | 16:34:04 |
connor (burnt/out) (UTC-8) | That’s partly why my packaging of Onnxruntime involves rewriting some of their CMake files | 16:34:35 |
connor (burnt/out) (UTC-8) | I also love that onnx by default builds with ONNX_ML=0 (disabling old APIs in favor of new ones), but various projects depend on it being set to one value or the other, so you could very easily end up with two copies of onnx, each configured with a different value for ONNX_ML. | 16:36:22 |
connor (burnt/out) (UTC-8) | God what a nightmare | 16:36:36 |
connor (burnt/out) (UTC-8) | We should also update OpenCV at some point to 4.10 if we haven’t already so it can build with CUDA 12.4+ | 16:38:16 |
connor (burnt/out) (UTC-8) | Oh! Unrelated but this was a cute change I made that I quite like: https://github.com/ConnorBaker/cuda-packages/blob/c81a6595f07456c6cc34d8976031c4fa972a741f/cudaPackages-common/backendStdenv.nix#L36
Sets some defaults for the CUDA stdenv and adds a name prefix, similar to what the Python packaging does, for more descriptive store paths | 16:40:07 |
Gaétan Lepage | Thanks for taking the time to look at it and explain all of this ! | 21:26:24 |
| 25 Oct 2024 |
| * connor (burnt/out) (UTC-8) makes a sad noise | 11:51:25 |
connor (burnt/out) (UTC-8) | https://gist.github.com/ConnorBaker/6c9c522d46e4244eb33d2aad94c753b0 | 11:51:27 |
| 26 Oct 2024 |
Gaétan Lepage | 🥲 | 20:34:04 |
Gaétan Lepage |  Download clipboard.png | 20:34:07 |