#10716: Adding a weighted_degree function to Singular multivariate polynomials
-------------------------------------+-------------------------------------
Reporter: jsrn | Owner: AlexGhitza
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-6.3
Component: algebraic | Resolution:
geometry | Merged in:
Keywords: multivariate | Reviewers: Marshall Hampton
polynomials,degree,Singular | Work issues:
Authors: Johan S. R. | Commit:
Nielsen, Luis Felipe Tabera | 1036282f26205edced3fe67a6ba8f5a8969feb59
Alonso | Stopgaps:
Report Upstream: N/A |
Branch: |
u/lftabera/weighted_degree |
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by malb):
{{{#!python
if self.is_zero():
#Corner case, note that the degree of zero is a python int
return -1
}}}
Why not fix the bug?
{{{#!python
weigths = map(int, weights)
}}}
Why {{{int}}} and not {{{ZZ}}}?
{{{#!python
cdef int deg = -1
cdef int n = self.parent().ngens()
cdef int i, l
}}}
This is sets you up for integer overflows, why not avoid them by using
{{{ZZ}}}? The function is going to be slow in any case.
Also, doesn't Singular have an internal function which would allow to
calculate this?
--
Ticket URL: <http://trac.sagemath.org/ticket/10716#comment:19>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
