| 7 May 2025 |
 connor (he/him) (UTC-7) | Okay, should be good to go assuming CI passes | 18:24:37 |
| connor (he/him) (UTC-7) | To recap, if anyone has spare cycles to review:
- https://github.com/NixOS/nixpkgs/pull/385960 -- very large, lots of bash, but lots of unit tests
- https://github.com/NixOS/nixpkgs/pull/404686 -- very small, no rebuilds
- https://github.com/NixOS/nixpkgs/pull/404973 -- ~900 line diff, no rebuilds outside of CUDA packages, largely just re-organizing to minimize the diff when landing setup hook changes
The setup hook changes (working on a PR) will need all three merged.
| 18:28:26 |
 SomeoneSerge (Ever OOMed by Element) | Let's start by merging packagesFromDirectoryRecursive | 18:39:44 |
| SomeoneSerge (Ever OOMed by Element) | Hmm can't we somehow adopt ci/eval and .github/workflows/eval.yml to evaluate, say, release-cuda.nix for PRs labeled "cuda" | 18:47:10 |
| SomeoneSerge (Ever OOMed by Element) | (with builds and substitutions disabled) | 18:47:28 |
| SomeoneSerge (Ever OOMed by Element) | connor (he/him) (UTC-7): feel like scratching an itch... git mv pkgs/development/cuda{-modules,} | 19:02:48 |
| connor (he/him) (UTC-7) | I kept cuda-modules just because it seems everything else in development follows that naming scheme, but no real reason to keep it that way, I suppose | 19:38:14 |
| SomeoneSerge (Ever OOMed by Element) |
everything else in development follows that naming scheme
No idea why, but whatever
| 19:39:25 |
| SomeoneSerge (Ever OOMed by Element) | connor (he/him) (UTC-7): your shims are basically fixups for manifests aren't they? | 20:03:39 |
| connor (he/him) (UTC-7) | Yep! | 20:04:17 |
| SomeoneSerge (Ever OOMed by Element) | Now I want (aside from the stuff already listed in TODO): snapshots of nixpkgs attribute trees AND snapshots of callable arguments' functionArgs | 20:42:24 |
| connor (he/him) (UTC-7) | There's a builtin for functionArgs which is probably a little more in-depth than the __functionArgs I used: https://github.com/NixOS/nix/blob/1e822bd4149a8bce1da81ee2ad9404986b07914c/src/libexpr/primops.cc#L3135 | 20:47:12 |
| SomeoneSerge (Ever OOMed by Element) | There's lib.functionArgs too | 20:48:11 |
|  kaya changed their profile picture. | 21:21:02 |
|  Tristan Ross joined the room. | 21:46:04 |
| Tristan Ross | I have joined here because I may do CUDA stuff with my Ampere Altra Max M128-26 and RTX 5070. | 21:46:43 |
 hacker1024 | Ooh is that SBSA then? Good luck lol | 21:47:47 |
| Tristan Ross | In reply to @hacker1024:matrix.org
Ooh is that SBSA then? Good luck lol Yep, SBSA and ACPI | 21:49:07 |
| SomeoneSerge (Ever OOMed by Element) | Isn't this going to be like literally the first time anybody actually uses sbsa version of cuda in Nixpkgs? | 21:49:09 |
| Tristan Ross | In reply to @ss:someonex.net
Isn't this going to be like literally the first time anybody actually uses sbsa version of cuda in Nixpkgs? Maybe | 21:49:28 |
| Tristan Ross | I built a bunch of CUDA stuff that @[connor (he/him) (UTC-7)] has on Saturday | 21:50:07 |
| Tristan Ross | There's a lot of things broken lol. | 21:50:17 |
| connor (he/him) (UTC-7) | FWIW, I was able to build and train an MLP mixer model this weekend on a GH200 through Lambda Labs using my repo, so proper support is at least in progress lol | 22:04:12 |
| SomeoneSerge (Ever OOMed by Element) | Oh riiight we had multiple cudnns in each package sets and applying fixups to all of them 😅 | 22:25:08 |
| connor (he/him) (UTC-7) | Oh god right | 22:32:51 |
| SomeoneSerge (Ever OOMed by Element) | Couple days to the feature freeze, perfect time to remove some public attributes | 22:33:42 |
| connor (he/him) (UTC-7) | Welp, time to figure out how I managed to break the multiplexing builder | 22:34:16 |
| connor (he/him) (UTC-7) | derp | 22:40:42 |
| connor (he/him) (UTC-7) | diff --git a/pkgs/top-level/cuda-packages.nix b/pkgs/top-level/cuda-packages.nix
index fc4beb3cea0d..a4a8086c7bf1 100644
--- a/pkgs/top-level/cuda-packages.nix
+++ b/pkgs/top-level/cuda-packages.nix
@@ -40,7 +40,7 @@ let
# Utilities
mkVersionedPackageName =
- name: version: "name_${lib.replaceStrings [ "." ] [ "_" ] (lib.versions.majorMinor version)}";
+ name: version: name + "_" + lib.replaceStrings [ "." ] [ "_" ] (lib.versions.majorMinor version);
in
{
config,
| 22:40:45 |
| connor (he/him) (UTC-7) | SomeoneSerge (UTC+U[-12,12]): you okay with me merging https://github.com/NixOS/nixpkgs/pull/405031? | 22:51:56 |
