Skip to content
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

Upgrading pyinstaller #2851

Closed
wants to merge 6 commits into from
Closed

Upgrading pyinstaller #2851

wants to merge 6 commits into from

Conversation

wpotrzebowski
Copy link
Contributor

@wpotrzebowski wpotrzebowski commented Apr 11, 2024

Description

It seems that we need to use earlier version scipy (1.12), to make MAC installer working.
As always in such case, the fix should be removed once proper bug fix from scipy is provided.
Addressing #2846

@wpotrzebowski wpotrzebowski marked this pull request as ready for review April 12, 2024 06:01
@wpotrzebowski wpotrzebowski changed the title Downgrading scipy to check if OSX installer runs again Downgrading scipy so that OSX installer runs again Apr 13, 2024
@butlerpd
Copy link
Member

Does this not affect windows as well? (See Julius' comment in #2846). What about release 6.0? Does it not need t be downgraded as well?

@butlerpd
Copy link
Member

As suggested by @andyfaff in #2846, this is apparently as PyInstaller issue (see stack overflow discussion started 8 days ago) more than a scipy issue and should be fixed by upgrading to PyInstaller 6.6.0. As reported in the PyInstaller 6.6.0 Changelog published Friday:

Hooks

  • Update PySide6.Qt3DRender hook for compatibility with PySide6 6.7.0 (add hidden import for PySide6.QtOpenGL module). (#8404)

  • Update scipy.special._ufuncs hook for compatibility with SciPy 1.13.0 (add scipy.special._cdflib to hidden imports). (#8394)

Suggest we do not revert scipy but instead upgrade PyInstaller. This should be done Post Haste as it affects ALL builds (window and mac, release and main)

@wpotrzebowski
Copy link
Contributor Author

Ok, indeed it seems that pyinstaller version with the fix was released 3 days ago. I noticed that we have quite discrepancy between main and release_6.0.0

For release 6.0.0

       - name: Install utilities to build installer
        if: ${{ matrix.installer && startsWith(matrix.os, 'windows') }}
        run: |
          python -m pip install pyinstaller==5.13.2
      - name: Install utilities to build installer
        if: ${{ matrix.installer && !startsWith(matrix.os, 'windows') }}
        run: |
          python -m pip install pyinstaller==6.3.0

while main installs python -m pip install pyinstaller==5.13.2 for all platforms. However, when I tried to upgrade Windows in January, there were some issues and we decided to keep with the older version.
So I agree we should upgrade pyinstaller but it may not be straight forward.

@wpotrzebowski wpotrzebowski changed the title Downgrading scipy so that OSX installer runs again Upgrading pyinstaller Apr 17, 2024
@wpotrzebowski
Copy link
Contributor Author

It seems to be solving the issue with MAC

@wpotrzebowski
Copy link
Contributor Author

Closing as won't be able to resolve Windows at the minute

@wpotrzebowski wpotrzebowski deleted the osx_installer_fix branch April 17, 2024 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants