Skip to content
This repository has been archived by the owner on Nov 28, 2024. It is now read-only.

tensorflowWithoutCuda #16

Open
hacker1024 opened this issue Dec 18, 2023 · 3 comments
Open

tensorflowWithoutCuda #16

hacker1024 opened this issue Dec 18, 2023 · 3 comments

Comments

@hacker1024
Copy link

The tensorflowWithoutCuda derivation changes slightly when config.cudaSupport is enabled, as some of its dependencies use CUDA.

Can this variant be added as a CI job?

@SomeoneSerge
Copy link
Owner

Sure, can you please submit the patch? Can you also open a nixpkgs issue about the attribute not being stable under config.cudaSupport? It shouldn't change, ideally

@hacker1024
Copy link
Author

I believe it should change, as TensorFlow has dependencies that use CUDA. If it were to override those, it might lead to conflicts with other packages in a larger CUDA-enabled Python environment.

@SomeoneSerge
Copy link
Owner

Yes, it's likely that tensorflowWithoutCuda would end up being non-composable with many other python packages. It might still work for python "applications". It'll also become less of a concern once adisbladis's work (requiredPythonModules) gets merged. We might want to reconsider whether tensorflowWithoutCuda is making a promise it can't actually keep.

All of those are nixpkgs concerns

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants