Opened 8 years ago

Closed 8 years ago

#11879 closed enhancement (fixed)

Intersection for more than two ideals

Reported by: ranosch Owned by: malb
Priority: trivial Milestone: sage-5.0
Component: commutative algebra Keywords: singular intersection sd34 sd35
Cc: PolyBoRi, burcin Merged in: sage-5.0.beta1
Authors: Niels Ranosch Reviewers: Julian Rueth
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #9138, #11900, #11115, #11068, #11761, #4539 Stopgaps:

Description

Currently, sage.rings.polynomial.multi_polynomial_ideal.MPolynomialIdeal_singular_repr.intersection() only takes one argument, while the singular interpreter accepts more.

Attachments (1)

trac-11879_intersection-several-ideals.patch​ (2.5 KB) - added by ranosch 8 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 8 years ago by ranosch

  • Status changed from new to needs_review

comment:2 Changed 8 years ago by saraedum

  • Status changed from needs_review to positive_review

applies to sage-4.8.alpha3 and all doctests pass.

comment:3 Changed 8 years ago by saraedum

  • Keywords sd35 added

comment:4 Changed 8 years ago by jdemeyer

  • Milestone changed from sage-4.8 to sage-5.0

comment:5 Changed 8 years ago by jdemeyer

  • Dependencies set to #9138, #11900, #11115, #11068, #11761, #4539
  • Status changed from positive_review to needs_work
  • Work issues set to rebase

This patch conflicts with #4539 and should be rebased.

Changed 8 years ago by ranosch

comment:6 Changed 8 years ago by ranosch

  • Status changed from needs_work to needs_review

Rebased on 4.7.2 with these patches applied: 

trac4539_libplural_rel11761.patch
trac4539_pickling_rel10903.patch
trac4539_normal_forms_rel10903.patch
trac4539_fix_docs_rel10903.patch

The "conflict" was a blank line.

comment:7 Changed 8 years ago by jdemeyer

  • Reviewers set to Julian Rueth
  • Status changed from needs_review to positive_review
  • Work issues rebase deleted

If you just did a simple rebase, there is no need to re-review it. You can set back positive review yourself.

comment:8 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.0.beta1
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.