Hi,
I now could remove all the blacklisted domains in the aldor-interface.
https://github.com/hemmecke/fricas/tree/fix-blacklisted-domains
In fact, aldor helped to discover a bug in the two spad files, that
actually the SPAD compiler should have refused to compile.
See explanation in the commit message... (attaced at the end of this mail)
https://github.com/hemmecke/fricas/commit/e68b728e0a5d7491e07f4a45db82e5cf2acf02cd
Although I personally don't like a type that involves #vl, a proper fix
would involve a change in the inferface of
DistributedMultivariatePolynomial. That seems to be too involved for the
moment, and maybe isn't wanted by other people.
I would like to make 3 commits as given below. I only have a ChangeLog
entry in the last commit, but I hope it's OK that way.
May I commit?
Ralf
=============================================================
commit 303577fa7632cfe6fc9f14d54ce5611e50043e14 (HEAD,
origin/fix-blacklisted-do
Author: Ralf Hemmecke <[email protected]>
Date: Sat Dec 31 17:25:22 2011 +0100
forget about blacklisted domains
ChangeLog | 6 ++++++
src/aldor/Makefile.in | 9 ---------
2 files changed, 6 insertions(+), 9 deletions(-)
commit e68b728e0a5d7491e07f4a45db82e5cf2acf02cd
Author: Ralf Hemmecke <[email protected]>
Date: Sun Feb 5 23:57:54 2012 +0100
fix: remove wrong parameter nv
IdealDecompositionPackage and QuasiAlgebraicSet2 used to require two
parameters where the first parameter was a list of variables and the
second parameter was supposed to be the length of this list.
This additional parameter for the list length was seemingly necessary
because of a weakness of the compiler. It no longer is.
In fact, that parameter was wrong conceptually and compilation should
have been rejected by the SPAD compiler. Formerly we had
F ==> Fraction Integer
Var ==> OrderedVariableList vl
Expon ==> DirectProduct(nv,NNI)
Dpoly ==> DistributedMultivariatePolynomial(vl,F)
QALG ==> QuasiAlgebraicSet(F, Var, Expon, Dpoly)
The 4 parameter of QuasiAlgebraicSet is required to be of type
PolynomialCategory(F,Expon,Var). However, the type of
DistributedMultivariatePolynomial(vl,F) is
PolynomialCategory(F,DirectProduct(#vl,NonNegativeInteger),
OrderedVariableList(vl)) with
reorder: (%,List Integer) -> %
This was a mismatch in the third parameter, namely the provided
DirectProduct(nv,NNI) vs. the expected
DirectProduct(#vl,NonNegativeInteger)
src/algebra/idecomp.spad.pamphlet | 6 ++----
src/algebra/qalgset.spad.pamphlet | 7 +++----
2 files changed, 5 insertions(+), 8 deletions(-)
commit 9449d68bfd108e70d85378c5d5c3d833f802d8d2
Author: Ralf Hemmecke <[email protected]>
Date: Sun Feb 5 21:51:28 2012 +0100
fix definition of RationalNumber
src/algebra/random.spad.pamphlet | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
--
You received this message because you are subscribed to the Google Groups "FriCAS -
computer algebra system" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/fricas-devel?hl=en.