!eWOErHSaiddIbsUNsJ:nixos.org

NixOS CUDA

288 Members
CUDA packages maintenance and support in nixpkgs | https://github.com/orgs/NixOS/projects/27/ | https://nixos.org/manual/nixpkgs/unstable/#cuda58 Servers

Load older messages


SenderMessageTime
11 Mar 2025
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • adding tensorrt.hydraPlatforms = none

?

As I've mentioned, you can see the exact code I am talking about here.

19:22:44
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

As I've mentioned, you can see the exact code I am talking about here.

19:23:46
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

We can keep the Qt 5 missing (<2022.4.2.1)/Qt 6 missing (>=2022.4.2.1) as brokenConditions, because they aren't currently producing eval errors and it makes sense that we want to detect these conditions in CI.

As I've mentioned, you can see the exact code I am talking about here.

19:25:41
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

We can keep the Qt 5 missing (<2022.4.2.1)/Qt 6 missing (>=2022.4.2.1) as brokenConditions, because they aren't currently producing eval errors and it makes sense that we want to detect these conditions in CI.

As I've mentioned, you can see the exact changes I am talking about here.

19:25:54
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

We can keep the Qt 5 missing (<2022.4.2.1)/Qt 6 missing (>=2022.4.2.1) as brokenConditions, because they aren't currently producing eval errors and it makes sense that we want to detect these conditions in CI.

You can see the exact changes I am talking about here.

19:26:07
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

change brokenConditions to badPlatformsConditions

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

We can keep the Qt 5 missing (<2022.4.2.1)/Qt 6 missing (>=2022.4.2.1) as brokenConditions, because they aren't currently producing eval errors and it makes sense that we want to detect these conditions in CI.

You can see the exact changes I am talking about here.

19:28:07
@ruroruro:matrix.orgruro *

I just wanted to make sure. You left this comment on the release-cuda issue:

change brokenConditions to badPlatformsConditions

H'm well it seems to make sense at least for

nix eval nixpkgs#cudaPackages.cudnn_8_0.brokenConditions
{ "CUDA version is too new" = true; ... }

Did I understand correctly, that you are okay with

  • moving cudnn.brokenConditions."CUDA version is too old/new" to badPlatformsConditions
  • moving nsight_systems.brokenConditions."CUDA too old (<11.8)" to badPlatformsConditions
  • moving nvidia_driver.brokenConditions."Package is not supported; use drivers from linuxPackages" to badPlatformsConditions
  • setting tensorrt.hydraPlatforms = none (because it can't be built on Hydra)

?

You can see the exact changes I am talking about here.

We can keep the Qt 5 missing (<2022.4.2.1) / Qt 6 missing (>=2022.4.2.1) as brokenConditions, because they aren't currently producing eval errors and it makes sense that we want to detect these conditions in CI.

19:28:36
@ss:someonex.netSomeoneSerge (back on matrix) ruro: this sounds great! 22:02:03
@adam_neverwas:matrix.orgAdam Neverwas removed their display name adam_neverwas.23:24:15
@gumby0811:matrix.orggumby0811 set a profile picture.23:38:53
12 Mar 2025
@ruroruro:matrix.orgruro Any preferences on the alternative name for evalPackageSetPlatforms? ))) 00:42:04
@ruroruro:matrix.orgruro I am leaning towards packageSetPlatforms... 00:42:18
@ruroruro:matrix.orgruro * I am leaning towards just packageSetPlatforms... 00:42:32
@ss:someonex.netSomeoneSerge (back on matrix) stick: I was going through the missed notifications today and saw a whole bunch of merged cuda-related fixes and refactorings. That was nice:) 00:43:02
@ss:someonex.netSomeoneSerge (back on matrix) Well, looking at the file again, we have packageSets, so the platforms are also of multiple package set<b>s</b>, but that's inconvenient to type, so maybe psPlatforms? 00:51:57
@ss:someonex.netSomeoneSerge (back on matrix) And comparing to pkgs/top-level/release.nix they introduce packageJobs, but then just use the in-place map for pkgs 00:52:26
@ruroruro:matrix.orgruro Maybe packagePlatforms then? I mean... the attrset specifies the platforms for each package, not for each package set. 00:54:15
@ruroruro:matrix.orgruro * Maybe packagePlatforms then? I mean... this attrset specifies the platforms for each package, not for each package set. 00:54:23
@ruroruro:matrix.orgruro It's a bit unfortunate that packagePlatforms is a function in pkgs/top-level/release.nix, but eh... 00:56:23
@ruroruro:matrix.orgruroNvm00:56:47
@ruroruro:matrix.orgruro It's also a function inherited from release-lib 00:57:04
@ruroruro:matrix.orgruro * packagePlatforms is also a function inherited from release-lib 00:57:19
@ss:someonex.netSomeoneSerge (back on matrix)well, platforms for entire sets of packages, and the reason that stuff is in a separate variable is because individual packages are handled differently00:57:32
@ruroruro:matrix.orgruroYeah...00:57:50
@ruroruro:matrix.orgruroDamn, naming things is hard00:58:00
@ruroruro:matrix.orgruro platformsForPackageSets? 00:58:36
@ss:someonex.netSomeoneSerge (back on matrix)Sure. Or... just keep the function application without a name xD00:59:21
@ruroruro:matrix.orgruro Or alternatively, we could rename packageSets to cudaPackageSets and then rename evalPackageSetPlatforms to cudaPackagePlatforms. 00:59:21
@ss:someonex.netSomeoneSerge (back on matrix)I guess the thought was we might start being less picky and start adding either subscopes?..01:01:12
@ruroruro:matrix.orgruro On one hand this isn't very future proof because of potential future rocm and mkl support. On the other hand, the file is already called release-cuda, not release-unfree or something 01:01:15

Show newer messages


Back to Room ListRoom Version: 9