Opened 9 years ago
Closed 9 years ago
#14587 closed defect (fixed)
Gröbner bases via Macaulay2 broken
Reported by: | Martin Albrecht | Owned by: | William Stein |
---|---|---|---|
Priority: | major | Milestone: | sage-5.10 |
Component: | interfaces | Keywords: | |
Cc: | wstein | Merged in: | sage-5.10.rc0 |
Authors: | Martin Albrecht | Reviewers: | Volker Braun |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
this shouldn't happen:
P.<a,b,c> = PolynomialRing(ZZ,3) I = sage.rings.ideal.Katsura(P,3) # regenerate to prevent caching I.groebner_basis('macaulay2:gb') # optional - macaulay2 TypeError: _groebner_basis_macaulay2() got an unexpected keyword argument 'prot'
Apply
Attachments (2)
Change History (10)
Changed 9 years ago by
Attachment: | trac_14587_m2.patch added |
---|
comment:1 Changed 9 years ago by
Description: | modified (diff) |
---|---|
Reviewers: | → Volker Braun |
comment:2 Changed 9 years ago by
I cannot test the fixes (Debian doesn't have M2). I can address that one, but it's probably quicker if you or someone with M2 fixes them?
comment:3 Changed 9 years ago by
Status: | new → needs_review |
---|
Ok, done. Positive review to your half of the patch...
comment:4 Changed 9 years ago by
Status: | needs_review → positive_review |
---|
works on Sage 5.9 with system-wide M2 1.6 (OSX 10.6.8)
sage -bt --optional=sage,macaulay2 --long sage/rings/ ... All tests passed! ---------------------------------------------------------------------- Total time for all tests: 862.0 seconds cpu time: 774.6 seconds cumulative wall time: 828.1 seconds
comment:5 Changed 9 years ago by
Status: | positive_review → needs_info |
---|
Which patch(es) should be applied?
comment:6 Changed 9 years ago by
Description: | modified (diff) |
---|---|
Status: | needs_info → needs_review |
comment:7 Changed 9 years ago by
Status: | needs_review → positive_review |
---|
comment:8 Changed 9 years ago by
Merged in: | → sage-5.10.rc0 |
---|---|
Resolution: | → fixed |
Status: | positive_review → closed |
Note: See
TracTickets for help on using
tickets.
I can confirm that the fix works (with Macaulay2 installed from Fedora).
There are some failing doctests, for example
Can you clean those up, too?