-
-
Notifications
You must be signed in to change notification settings - Fork 19.2k
python3Packages.torch: package NNPACK separately #288140
Copy link
Copy link
Open
Labels
2.status: stalehttps://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.mdhttps://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md6.topic: pythonPython is a high-level, general-purpose programming language.Python is a high-level, general-purpose programming language.
Metadata
Metadata
Assignees
Labels
2.status: stalehttps://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.mdhttps://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md6.topic: pythonPython is a high-level, general-purpose programming language.Python is a high-level, general-purpose programming language.
Fields
Give feedbackNo fields configured for issues without a type.
Issue description
Torch needs to be rid of
fetchSubmodules = true(they prevent dependency injection; #239291, #277789, #276585), but in particular in #285249 we encountered the need to patch and configure NNPACK (unconditionally downloads dependencies from the internet without any error handling, fails configuration phase by running python with an invalid PYTHONPATH). Doing so inside the torch derivation would be cumbersome and doesn't scale (many more git-submodule time bombs). We should handle NNPACK once, without adding complexity to the torch expression. We should also ensure we can passcmakeFlagsto torch (currently obstructed by thesetup.pywrapper)References
#239211
Technical details
Please run
nix-shell -p nix-info --run "nix-info -m"and paste the result.