-
-
Notifications
You must be signed in to change notification settings - Fork 516
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
inclusion of the ore_algebra within Sage #16008
Comments
comment:2
The link is http://www.risc.jku.at/research/combinat/software/ore_algebra/ore_algebra-0.1.spkg and it installs out of the box. |
comment:3
Hmm, what is it that needs review here? |
comment:4
Well I thought I can't set both review and positive... but since you reviewed it too, I'll just go ahead. Thanks. |
comment:5
I don't understand: |
comment:6
This ticket is closed when ore_algebra is included as optional package. Any other code you need to use the package should go in another ticket. |
comment:7
Replying to @rwst:
But it isn't, is it?
Or did you forget to attach a branch to this ticket? |
comment:8
But why? It's all included in the SPKG and works. Also, the docs state: |
comment:9
Replying to @rwst:
Sure, |
comment:10
I do not suggest to close the ticket as wontfix. I never said that. What I expect is that someone sets positive, and since this ticket is in Again, this ticket is in |
comment:11
Replying to @rwst:
Sorry, I never reviewed optional packages before, and I thought the idea was to use the ticket to review that the package worked well once all that was done, not to decide whether to make it an optional package at all. But if positive review only means "yes, this should be an optional package," then I agree with you. |
comment:13
Thanks. John Cremona clarified the procedure in https://groups.google.com/forum/#!topic/sage-devel/jopuoWO1Fvk New commits:
|
Commit: |
Reviewer: Marc Mezzarobba, Ralf Stephan |
comment:14
New-style optional packges must be tar.gz/tar.bz. We can include the existing spkg as optional, but then we don't need anything from the git branch. |
comment:15
So is there anything missing for this ticket? |
comment:16
The attached git branch is incomplete and there is no tarball. Is the ticket about updating the legacy spkg or creating a new-style optional package? I'm confused... |
comment:17
This is about taking the published SPKG and using it to create an optional package. You said elsewhere that a functional SPKG suffices for optional package creation. I created the branch before I knew that. |
comment:18
Harald: can you mirror it? |
Changed branch from u/rws/inclusion_of_the_ore_algebra_within_sage to none |
Changed commit from |
This comment has been minimized.
This comment has been minimized.
comment:19
yes, done. ore_algebra is on the servers |
the group of Manuel Kauers at RISC develops the
ore_algebra
package(http://www.risc.jku.at/research/combinat/software/ore_algebra/index.php) which is similar to the
Maple gfun package (but more powerful since it works in more general rings), and will be extended to
the multivariate case. A small example with version 0.1:
New optional spkg: http://www.risc.jku.at/research/combinat/software/ore_algebra/ore_algebra-0.1.spkg
CC: @mkauers @haraldschilly
Component: packages: optional
Keywords: gfun Mgfun Ore guessing
Reviewer: Marc Mezzarobba, Ralf Stephan
Issue created by migration from https://trac.sagemath.org/ticket/16008
The text was updated successfully, but these errors were encountered: