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

pythran 0.17.0, switch to wheel #39652

Merged
merged 5 commits into from
Mar 22, 2025
Merged

pythran 0.17.0, switch to wheel #39652

merged 5 commits into from
Mar 22, 2025

Conversation

dimpase
Copy link
Member

@dimpase dimpase commented Mar 8, 2025

Previous version getting obsolete, also, a
preparation for scipy update

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

@dcoudert
Copy link
Contributor

I still get

[xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx]
ERROR [transfer|run:135]: [Errno socket error] [Errno 404] Not Found: '//linorg.usp.br/sage/spkg/upstream/pythran/pythran-0.17.0-py3-none-any.whl'
Attempting to download from https://files.pythonhosted.org/packages/packages/py3/p/pythran/pythran-0.17.0-py3-none-any.whl
[xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx]
ERROR [transfer|run:135]: [Errno socket error] [Errno 404] Not Found: '//files.pythonhosted.org/packages/packages/py3/p/pythran/pythran-0.17.0-py3-none-any.whl'
Traceback (most recent call last):
  File "/usr/lib64/python3.12/urllib/request.py", line 1788, in open
    return getattr(self, name)(url)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/urllib/request.py", line 2002, in open_https
    return self._open_generic_http(self._https_connection, url, data)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/urllib/request.py", line 1957, in _open_generic_http
    return self.http_error(
           ^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/urllib/request.py", line 1979, in http_error
    return self.http_error_default(url, fp, errcode, errmsg, headers)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/download/transfer.py", line 107, in http_error_default
    raise DownloadError(errcode, errmsg, url)
sage_bootstrap.download.transfer.DownloadError: [Errno 404] Not Found: '//files.pythonhosted.org/packages/packages/py3/p/pythran/pythran-0.17.0-py3-none-any.whl'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/tarball.py", line 185, in download
    Download(url, destination).run()
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/download/transfer.py", line 137, in run
    raise error
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/download/transfer.py", line 128, in run
    filename, info = opener.retrieve(
                     ^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/urllib/request.py", line 1822, in retrieve
    fp = self.open(url, data)
         ^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.12/urllib/request.py", line 1794, in open
    raise OSError('socket error', msg) from msg
OSError: [Errno socket error] [Errno 404] Not Found: '//files.pythonhosted.org/packages/packages/py3/p/pythran/pythran-0.17.0-py3-none-any.whl'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/dcoudert/sage/build/bin/sage-package", line 40, in <module>
    run()
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/cmdline.py", line 538, in run
    app.download_cls(*args.package_class,
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/app.py", line 337, in download_cls
    pc.apply(download_with_args)
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/expand_class.py", line 126, in apply
    function(package_name, *args, **kwds)
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/app.py", line 329, in download_with_args
    self.download(package, allow_upstream=allow_upstream)
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/app.py", line 315, in download
    package.tarball.download(allow_upstream=allow_upstream)
  File "/home/dcoudert/sage/build/bin/../sage_bootstrap/tarball.py", line 187, in download
    raise FileNotMirroredError('tarball does not exist on mirror network and neither at the upstream URL')
sage_bootstrap.tarball.FileNotMirroredError: tarball does not exist on mirror network and neither at the upstream URL
************************************************************************
Error downloading tarball of pythran
************************************************************************

@dimpase
Copy link
Member Author

dimpase commented Mar 11, 2025

@dcoudert - apologies, should be fixed now. Such an error-prone stuff...

Copy link
Contributor

@dcoudert dcoudert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@dcoudert
Copy link
Contributor

Actually, there is something I don't understand: this PR bump pythran to version 0.17.0 but it indicates in file build/pkgs/pythran/version_requirements.txt : pythran>=0.14.0,<0.17.0.
In #39655, we don't see any modification of file build/pkgs/pythran/version_requirements.txt which still indicates pythran >=0.12.1.

@dimpase
Copy link
Member Author

dimpase commented Mar 11, 2025

Actually, there is something I don't understand: this PR bump pythran to version 0.17.0 but it indicates in file build/pkgs/pythran/version_requirements.txt : pythran>=0.14.0,<0.17.0. In #39655, we don't see any modification of file build/pkgs/pythran/version_requirements.txt which still indicates pythran >=0.12.1.

yes, your're right, it's fixed now.

I again failed to navigate the maze which package managers like pip do well, and is a perfect illustration of why I argued and argue a lot against vendoring Python packages. Because it forces a human to do the job in which automatic tools are much better than humans.

Why do I have to retrace by hand what's already done and dusted by maintainers of numpy+scipy, namely, figuring out what versions of tools they need?

Copy link
Contributor

@dcoudert dcoudert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

I agree with you that we should have a simpler way to update packages, and that we should use more system packages.

Copy link

Documentation preview for this PR (built with commit 631888e; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

vbraun pushed a commit to vbraun/sage that referenced this pull request Mar 19, 2025
sagemathgh-39652: pythran 0.17.0, switch to wheel
    
Previous version getting obsolete, also, a
preparation for scipy update


<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#39652
Reported by: Dima Pasechnik
Reviewer(s): David Coudert
vbraun pushed a commit to vbraun/sage that referenced this pull request Mar 19, 2025
sagemathgh-39655: update numpy to 2.2.3, scipy to 1.15.2
    
this is long overdue update.

It needs building  tools bumped, too, thus dependencies

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies
build tools need a bump
-  sagemath#39652 : pythran
- sagemath#39653 : ninja
-  sagemath#39654: pybind11
<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#39655
Reported by: Dima Pasechnik
Reviewer(s): David Coudert
vbraun pushed a commit to vbraun/sage that referenced this pull request Mar 22, 2025
sagemathgh-39655: update numpy to 2.2.3, scipy to 1.15.2
    
this is long overdue update.

It needs building  tools bumped, too, thus dependencies

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [x] The title is concise and informative.
- [x] The description explains in detail what this PR is about.
- [x] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation and checked the documentation
preview.

### ⌛ Dependencies
build tools need a bump
-  sagemath#39652 : pythran
- sagemath#39653 : ninja
-  sagemath#39654: pybind11
<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#39655
Reported by: Dima Pasechnik
Reviewer(s): David Coudert
@vbraun vbraun merged commit bff2e23 into sagemath:develop Mar 22, 2025
23 of 37 checks passed
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.

3 participants