| 9 May 2025 |
Kevin Mittman (UTC-8) | could you provide a pseudo-code form of what you are asking for? | 00:50:30 |
Kevin Mittman (UTC-8) | specifically trying to ask if there would be one or multiple versions of each component to satisfy the dependency closure? | 00:58:31 |
Guillaume Desforges | Hey! Has anyone tried to package torchcodec?
https://github.com/pytorch/torchcodec | 17:07:26 |
Guillaume Desforges | * Hey! Has anyone tried to package torchcodec?
https://github.com/pytorch/torchcodec
(sorry if that's the wrong channel) | 17:22:33 |
Guillaume Desforges | * Hey! Has anyone tried to package torchcodec?
https://github.com/pytorch/torchcodec
(sorry if that's the wrong channel, afaik it's ok since it's NVIDIA-related) | 17:22:48 |
Guillaume Desforges | * Hey! Has anyone tried to package torchcodec?
https://github.com/pytorch/torchcodec
(sorry if that's the wrong channel, afaik it's ok since it's torch/NVIDIA-related) | 17:23:13 |
Guillaume Desforges | * Hey! Has anyone tried to package torchcodec?
https://github.com/pytorch/torchcodec
(sorry if that's the wrong channel, afaik it's ok since it's torch/CUDA-related) | 17:23:25 |
connor (he/him) | :L on master:
nix-repl> legacyPackages.x86_64-linux.cudaPackages.cudnn.src
«derivation /nix/store/7gwjjblxcjxbbj8ry5zir6m35gks3aq6-cudnn-linux-aarch64-9.8.0.87_cuda12-archive.tar.xz.drv»
| 18:01:59 |
connor (he/him) | lmao somehow https://github.com/NixOS/nixpkgs/commit/9fd753ea84e5035b357a275324e7fd7ccfb1fc77 caused this | 18:09:44 |
connor (he/him) | I'll work on a fix shortly | 18:17:22 |
connor (he/him) | hmm, in hindsight the removed and changed packages listed here were a red flag: https://github.com/NixOS/nixpkgs/actions/runs/14912284892/attempts/1#summary-41889732827 | 18:18:44 |
connor (he/him) | Okay! https://github.com/NixOS/nixpkgs/pull/405707 should fix it --it also cleans up a fair amount of the logic. | 20:23:06 |
connor (he/him) | Also, if someone would mind reviewing https://github.com/NixOS/nixpkgs/pull/405664 | 21:02:13 |
connor (he/him) | Gaétan Lepage: any chance you'd be able to look at this? I'd like to get it merged soon since things depending on cuDNN (like PyTorch) are broken on x86_64-linux. | 21:22:03 |
Gaétan Lepage | I started to look at it but got scared by the nature of the diff | 21:22:53 |
Gaétan Lepage | Will look at it :) | 21:22:58 |
connor (he/him) | <3 | 21:23:01 |
connor (he/him) | If it helps at all, the evaluatedModules.config.${pname}.releases.${redistArch} attribute set we're indexing into is essentially just the releases.nix file each of the multiplexed packages provides (for example, here's cuDNN's: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/cuda-modules/cudnn/releases.nix). | 21:26:01 |
connor (he/him) | The call to lib.foldl' inside newestPackages traverses the list of package (the elements of the list of attribute sets underneath each architecture in releases.nix), keeping the latest copy of each major-minor version. | 21:28:59 |
Gaétan Lepage | Yes, I got a broad idea of what was happening | 21:30:20 |
Gaétan Lepage | Anyway, should be good to merge | 21:30:36 |
Gaétan Lepage | Subsidiary question: any reason for using lib.map instead of builtins.map? | 21:31:04 |
connor (he/him) | IIRC generally lib is preferred because it provides polyfills for functions which aren't available in older versions of Nix and Nixpkgs needs to be able to evaluate with very, very old versions of Nix | 21:37:38 |
| 10 May 2025 |
| @athyfr:matrix.org left the room. | 12:19:12 |
| 11 May 2025 |
connor (he/him) | Alrighty. Things for today:
- fixing Darwin eval since https://github.com/NixOS/nixpkgs/pull/405707 broke it (see my last comment for the fix)
- finishing up PR description for https://github.com/NixOS/nixpkgs/pull/405751
- publishing PR which introduces fix for package set leakage
- publishing PR which introduces pkgsCuda
- publishing PR which moves top-level/cuda-packages.nix to top-level/cuda-overlay.nix, allowing us to maintain CUDA package sets and top level CUDA attributes apart from top-level/all-packages.nix
| 14:18:46 |
connor (he/him) | Fix for darwin eval here: https://github.com/NixOS/nixpkgs/pull/406207 | 16:23:12 |
| 12 May 2025 |
connor (he/him) | SomeoneSerge (UTC+U[-12,12]): sorry for the late reply -- would you mind taking a look at https://github.com/NixOS/nixpkgs/pull/406207? I prefer the current iteration to the alternative you proposed | 00:02:32 |
SomeoneSerge (back on matrix) | Been afraid to open Element on my laptop, it keeps causing OMs... | 16:12:35 |
connor (he/him) | Does the web client cause the same issue? | 16:26:11 |
connor (he/him) | Split the cudaLib changes into its own PR since the title and previous branch name were misleading: https://github.com/NixOS/nixpkgs/pull/406531 | 18:35:18 |