Design #849
Cleanup MinPoly code
Description
Implement syntax MinPolyXX(f, x)
with f in R/I
(instead of current MinPolyXX(f, I, x)
)
Related issues
History
#1
Updated by Anna Maria Bigatti over 8 years ago
- Target version changed from CoCoALib-0.99560 to CoCoALib-0.99550 spring 2017
#2
Updated by Anna Maria Bigatti about 8 years ago
The current functions should be renamed MinPolyXXQuot(f, I, z)
#3
Updated by Anna Maria Bigatti about 8 years ago
The function MinPoly
(and MinPolyQuot
) should be straight calls to MinPolyDef/MinPolyDefQuot
#4
Updated by John Abbott almost 8 years ago
- Target version changed from CoCoALib-0.99550 spring 2017 to CoCoALib-0.99560
#5
Updated by John Abbott over 6 years ago
- Target version changed from CoCoALib-0.99560 to CoCoALib-0.99600
#6
Updated by John Abbott over 6 years ago
- Related to Feature #900: New function: MinPoly of RingElem in quotient ring added
#7
Updated by Anna Maria Bigatti almost 6 years ago
- Tracker changed from Feature to Design
- Target version changed from CoCoALib-0.99600 to CoCoALib-0.99650 November 2019
#8
Updated by John Abbott almost 5 years ago
- Target version changed from CoCoALib-0.99650 November 2019 to CoCoALib-0.99700
#9
Updated by John Abbott almost 5 years ago
- Related to Bug #1248: MinPolyQuot: guaranteed and ideal without GBasis added
#10
Updated by John Abbott over 4 years ago
Better to postpone this?
#11
Updated by Anna Maria Bigatti over 4 years ago
After so much time I think "Why bother?". Mathematically it is nice and clean, practically it is next to pointless, and would also be tedious to use.
We made it clean by calling it MinPolyQuot
. I think the only cleanup we could do is about renaming all internal functions with Quot
, but nothing more than that.
I even made an ISSAC tutorial about it, without missing it at all.
I suggest Reject.
#12
Updated by Anna Maria Bigatti over 4 years ago
- Target version changed from CoCoALib-0.99700 to CoCoALib-0.99800
#13
Updated by John Abbott over 2 years ago
- Target version changed from CoCoALib-0.99800 to CoCoALib-0.99850
#14
Updated by John Abbott 5 months ago
- Status changed from New to In Progress
- % Done changed from 0 to 10
In 8 years we have never really needed the proposed new API, so we may as well reject this issue (as Anna suggested 4 years ago).
HOWEVER it would be good to do some cleaning inside the file SparsePoltOps-MinpPoly.C
as there seems to be some cruft there.
#15
Updated by John Abbott 5 months ago
- Related to Feature #1103: Pseudo-zero-dim ideals added
#16
Updated by Anna Maria Bigatti 4 months ago
- Target version changed from CoCoALib-0.99850 to CoCoALib-0.99880