#12861: maximal_order of quaternion algebras should be more general
-------------------------------+--------------------------------------------
Reporter: daniels | Owner: daniels
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.3
Component: algebra | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: | Merged in:
Dependencies: #12860 | Stopgaps:
-------------------------------+--------------------------------------------
Comment (by daniels):
Thanks for testing the patch!
I just tried applying the patch from #12860 and after that the second
patch from this ticket to a fresh install of Sage 5.2rc0, and the tests
passed.
I also tried inserting your debug print:
{{{
print 'f', f, 'dif', min_v - valuation(p,2)
}}}
after the calls to `f = normalize_basis_at_p(e[1:], p)`, but wasn't able
to reproduce this error message.
I noticed that my line numbers are a bit different, i.e., a call to `f =
normalize_basis_at_p(e[1:], p)` sits at line 2744 for you, while this only
occurs on 2756 and 2787 for me.
Are there any other patches I should apply to debug this? (I have only
#12860 followed by patch.2 of this ticket).
I did however notice a typo in the docstring of `normalize_basis_at_p` and
have fixed thatin a new upload (``p \eq 2`` becomes ``p = 2``).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12861#comment:5>
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 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.
