#5513: [with patch, positive review] Enhanced support for number field unit
groups
---------------------------+------------------------------------------------
Reporter: cremona | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.4.1
Component: number theory | Keywords: number field unit group
---------------------------+------------------------------------------------
Comment(by cremona):
Replying to [comment:6 cremona]:
> Replying to [comment:5 fwclarke]:
> > A couple more things:
> > * two more replacements of `defining_polynomial` by
`absolute_polynomial` are needed.
> >
> > * units can have norm -1.
> >
> > With the third patch everything works.
>
> Thanks a lot -- for what it's worth, I am (more than) happy with the
third patch, also a little red-faced about the norms!
PS Francis, if yo ufancied looking at #5518 too you can have the
satisfaction of adding .norm() in a similar place. It's a lot simpler and
shorter than this one.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5513#comment:7>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" 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/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
