-
Notifications
You must be signed in to change notification settings - Fork 299
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
exceding NSIS 2Go limit ... #591
Comments
will remove Tensorflow, I'm afraid |
I am using ffmpeg every day, called by Python functions. I don't use the very old version (v3.2.4) in the latest WinPython beta. I am currently using v3.3.3, which has functionality I need. I plan to test a newer version (currently v3.4.1) with my code when I need new functionality or bug fixes. Thus, I need to manage which ffmpeg version is used by my code. Currently, I hard code the path to ffmpeg in my code. If I were to use ffmpeg found in the PATH instead, WinPython puts its old version in the PATH before ffmpeg installed for the system. Another solution for me is to delete the ffmpeg that comes with each installation of WinPython and change my code to use ffmpeg found in the PATH instead. But my code will break if I install a new WinPython and forget to delete its ffmpeg. Since ffmpeg is 38 MB, I am wondering if including it in WinPython is the best solution. Especially since many users have multiple versions of WinPython installed. Besides Tensorflow, other Python packages depend on ffmpeg, such as imageio and moviepy. For inexperienced Windows users, would it be better to give them instructions on where to download ffmpeg, how to copy to C:\Program Files\ffmpeg, and add this folder to the PATH (even outside WinPython)? This way, the WinPython project doesn't have to decide which version of ffmpeg is best for the user. Also, the download size is smaller for users who don't need ffmpeg. |
Dear @hiccup7 I agree ffmpeg":
I checked removing Tensorflows as a proven solution. |
removing ffmpeg seems sufficient for now. |
numpy is the required library for important python packages like pandas, matplotlib. May I vote for keeping it on the winpython. |
for sure I can't remove numpy-mkl. |
back to the limit.... let's try not doing jupyterlab build |
no change... need a bigger cut... next week:
not enough, next try by removing:
|
I have a work colleague who has been trying to use Tensorflow. On the surface, it seems easy to use. However, execution is extremely slow. There are lots of problems using the GPU. He is very frustrated. It is very difficult for each user to figure out exactly which versions of NVidia driver, Cuda driver, Python, etc. are needed to make it work (doesn't work with AMD GPU). If the user does find a combination that works, better not update anything for fear of breaking it. Numba looks better than Tensorflow in terms of being debugged and supported. Since Tensorflow is such a big download, I propose not to include it in WinPython. |
I anticipate I will be force to drop Tensorflow, I can't imagine JupyterLab not growing further in size. |
dropping Tensorflow might be a good choice since Tensorflow supports only Nvidia GPU and, compare with Jupyterlab package, I think that it is less important than the Jupyterlab which demonstrates useful in many cases. |
There is nevertheless a huge space taken per Numpy+mkl/Jupyterlab/Tensorflow, that feels "more than what they should need" to me. I would dream this three packages to take a look on their size metric. If Japan chainer was smaller, I would take this road, but it doesn't seem to. Matplotlib-2.2.0 is sligthly smaller than Matplotlib-2.1.2, so size inflation is not a curse. |
these 3 packages are the huge one and 2 out of 3 are very important packages for computer science and physical science. I hope they are getting smaller some days. |
maybe Microsoft will resolve our problem by including a Machine Learning bloc in default Windows : https://arstechnica.com/gadgets/2018/03/microsoft-pushing-machine-learning-as-big-developer-feature-for-next-windows-update/ |
and maybe good news, resolving all issues (too good to be true, I have to check):
so:
|
Wow, if it works, we would gain 225 Mo additional space! |
it works, so we can re-add back something dropped that was important for users:
Unless strong argument, I would keep the things as they are, as:
|
Tensorflow-1.6 is compiled for avx processors only ... problem |
Winpython installer gets to big for NSIS:
Internal compiler error #12345: error mmapping file (2125247030, 18568856) is out of range.
... need to sacrifice some package or change of toolchain
what TreeSise says:
The text was updated successfully, but these errors were encountered: