cysignals: Replace compile-time dependency on pari by run-time dispatch #32431
Comments
|
comment:2
Seems to work for me (when adding to |
|
comment:3
But currently cysignals own tests are failing. Will see tomorrow what is going on. |
|
Author: Jonathan Kliem |
|
comment:4
It looks like this can be solved with and New commits:
|
|
Work Issues: cypari2 and cysignals need new releases |
|
Branch: public/32431 |
|
Commit: |
|
Attachment: cypari2-2.1.3b1.tar.gz |
|
comment:5
Attachment: cysignals-1.11.0a0.dev0.tar.gz Branch pushed to git repo; I updated commit sha1. New commits:
|
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
|
comment:12
What's the status here? |
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Aug 27, 2024
<!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> - https://pypi.org/project/cypari2/2.2.0/ - https://github.com/sagemath/cypari2/blob/master/NEWS - Another step toward sagemath#32431 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] 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#38183 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
vbraun
pushed a commit
to vbraun/sage
that referenced
this issue
Aug 28, 2024
<!-- ^ Please provide a concise and informative title. --> <!-- ^ Don't put issue numbers in the title, do this in the PR description below. --> <!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method to calculate 1 + 2". --> <!-- v Describe your changes below in detail. --> <!-- v Why is this change required? What problem does it solve? --> <!-- v If this PR resolves an open issue, please link to it here. For example, "Fixes sagemath#12345". --> - https://pypi.org/project/cypari2/2.2.0/ - https://github.com/sagemath/cypari2/blob/master/NEWS - Another step toward sagemath#32431 ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> - [x] The title is concise and informative. - [ ] 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#38183 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Upstream issue:
sagemath/cysignals#125
CC: @kliem @tscrim @videlec @dimpase
Component: packages: standard
Work Issues: cypari2 and cysignals need new releases
Author: Jonathan Kliem
Branch/Commit: public/32431 @
0353619Issue created by migration from https://trac.sagemath.org/ticket/32431
The text was updated successfully, but these errors were encountered: