!eWOErHSaiddIbsUNsJ:nixos.org

NixOS CUDA

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

Load older messages


SenderMessageTime
4 Aug 2022
@winterqt:nixos.devWinter (she/her)(hi, just came here to read + respond to this.)03:28:52
@tpw_rules:matrix.orgtpw_ruleshey. i had previously sympathzied with samuela and like i said before had some of the same frustrations. i just edited my github comment to add "[CUDA] packages are universally complicated, fragile to package, and critical to daily operations. Nix being able to manage them is unbelievably helpful to those of us who work with them regularly, even if support is downgraded to only having an expectation of function on stable branches."03:29:14
@winterqt:nixos.devWinter (she/her)
In reply to @tpw_rules:matrix.org
i'm mildly peeved about a recent merging of something i maintain where i'm pretty sure the merger does not own the expensive hardware required to properly test the package. i don't think it broke anything but i was given precisely 45 minutes to see the notification before somebody merged it
ugh, 45 minutes? that's... not great. not to air dirty laundry but did you do what samuela did in the wandb PR and at least say that that wasn't a great thing to do? (not sure how else to word that, you get what i mean)
03:30:23
@tpw_rules:matrix.orgtpw_rulesno, i haven't yet, but i probably will03:31:03
@winterqt:nixos.devWinter (she/her) i admittedly did that with a PR once, i forget how long the maintainer was requested for but i merged it because multiple people reported it fixed the issue. the maintainer said "hey, don't do that" after and now i do think twice before merging. so it could help, is what i'm saying. 03:31:50
@tpw_rules:matrix.orgtpw_rulesi'm not sure what went wrong with the wandb PR anyway, i think it was just a boneheaded move on the maintainer's part03:32:10
@winterqt:nixos.devWinter (she/her)(it was also simple enough that it was fine and the maintainer said it looked good after)03:32:15
@tpw_rules:matrix.orgtpw_rules * i'm not sure what went wrong with the wandb PR anyway, i think it was just a boneheaded move on the merger's part03:32:19
@tpw_rules:matrix.orgtpw_rulesbut i thought most of the frustration was around packages which don't really involve CUDA breaking the fragile CUDA packages, and i'm not sure how the warning helps in this case. it's not like nixpkgs-review prints out the comments. maybe i'm wrong. but it is a legitimate problem03:34:19
@winterqt:nixos.devWinter (she/her)the frustration that i see is that people are touching packages that he maintains, am i missing further context here?03:35:09
@tpw_rules:matrix.orgtpw_rulesdid you ever see this? https://discourse.nixos.org/t/nixpkgss-current-development-workflow-is-not-sustainable/1874103:35:43
@winterqt:nixos.devWinter (she/her)oh yes i did03:35:49
@winterqt:nixos.devWinter (she/her)but that's not what the topic of this PR/the notice is, though?03:36:11
@winterqt:nixos.devWinter (she/her)this wouldn't help that03:36:14
@winterqt:nixos.devWinter (she/her)~~is that what you're saying and i'm just lagging behind~~03:36:27
@tpw_rules:matrix.orgtpw_rulesno it wouldn't, but it reads to me like that's the underlying problem and this is a manifestation which can be controlled more easily. not to put thoughts in people's head03:37:07
@winterqt:nixos.devWinter (she/her)

right

(what do you mean by that last sentence, you don't want to influence anyone's opinion on the matter by saying that?)

03:38:29
@tpw_rules:matrix.orgtpw_rulesi guess? it's my personal opinion and thought and i'd appreciate comment from the man himself 03:39:28
@tpw_rules:matrix.orgtpw_rulesi think i mixed my metaphors slightly. i don't intend to put words in his mouth03:40:00
@tpw_rules:matrix.orgtpw_rulesthere's also this: https://github.com/NixOS/nixpkgs/pull/18507803:42:14
@tpw_rules:matrix.orgtpw_rulesit's not really fair to characterize python general package updates as "breaking everything all over again" and the notice wouldn't have prevented it. it's just sort of life as being the center of a tangled web of dependencies03:43:13
@winterqt:nixos.devWinter (she/her)that should be something brought up with the python folks tbh03:43:18
@winterqt:nixos.devWinter (she/her)exactly03:43:20
@winterqt:nixos.devWinter (she/her)i'm sure it can be worked around somehow, or things can be put in place03:43:31
@winterqt:nixos.devWinter (she/her)the python folks are trying to keep up with something so fast moving and prone to breakage, there's only so much they can do without communication03:44:00
@tpw_rules:matrix.orgtpw_ruleslike sandro said in the issue, it feels more like there's a disconnect (which i don't know about) between the general python maintainers and cuda maintainers, and the best and most prompt solution might be to downgrade the service level03:44:06
@winterqt:nixos.devWinter (she/her)* the python folks are trying to keep up with something so fast moving and prone to breakage, there's only so much they can do03:44:09
@winterqt:nixos.devWinter (she/her)precisely yeah03:44:21
@tpw_rules:matrix.orgtpw_rulesi.e. focus on the packages around ZHF and accept that they're not stable and might take lots of work to get going if you follow unstable03:44:33
@winterqt:nixos.devWinter (she/her)(well the latter doesn't apply to samuela since he's a maintainer, so he kinda has to keep up in that regard)03:45:12

Show newer messages


Back to Room ListRoom Version: 9