Switch to slim Python Docker base image for TensorFlow docker builds and tidy up Python Dockerfile#437
Conversation
|
The |
|
That makes sense. How do we construct our own requirements.txt to go with it? Build the image, install the wheels then do |
|
You could do that, but I think I did it the other way around. I ran each example and installed the minimum required deps to make them work, and pinned just those to minor versions. We only care about the minor versions of things that are script facing, everything below was allowed to change for patch/security reasons. But happy to be persuaded otherwise. |
|
That's the ideal approach (albeit more costly in time). For now I won't worry about this as it sounds like the current |
Note that I currently have
to mimic what was done before. I'm not sure whether
--no-depsshould be used on both but the build breaks in that case. It's also the opposite approach to the PyTorch build (--no-depson the wheel installs and no--no-depson the requirements installs)... To be discussed...