#4000: Implement QQ['x'] via Flint ZZ['x'] + denominator
------------------------------+---------------------------------------------
Reporter: malb | Owner: somebody
Type: enhancement | Status: needs_review
Priority: blocker | Milestone: sage-4.6
Component: basic arithmetic | Resolution:
Keywords: | Author: Sebastian Pancratz, Martin
Albrecht
Upstream: N/A | Reviewer: John Cremona, Martin Albrecht,
Alex Ghitza, Harald Schilly
Merged: | Work_issues:
------------------------------+---------------------------------------------
Changes (by drkirkby):
* cc: pjeremy (added)
Comment:
Replying to [comment:123 mpatel]:
> The tests pass with a trial 4.6.alpha3 (which is probably the same as
alpha2 for this ticket) on sage.math, except for
> {{{
> #!python
> sage -t -long -force_lib
"devel/sage/sage/rings/number_field/number_field_ideal.py"
> **********************************************************************
> File
"/mnt/usb1/scratch/mpatel/apps/sage-4.6.a3/devel/sage/sage/rings/number_field/number_field_ideal.py",
line 194:
> sage: NumberField(x^2 + 1, 'a').ideal(7).__hash__()
> Expected:
> -9223372036854775779
> Got:
> -288230376151711715
> }}}
> On David Kirkby's !OpenSolaris machine hawk, I get
> {{{
> #!python
> sage -t -long -force_lib
"devel/sage/sage/rings/number_field/number_field_ideal.py"
> **********************************************************************
> File
"/export/home/buildbot/build/sage/hawk-1/hawk_full/build/sage-4.6.alpha3/devel/sage/sage/rings/number_field/number_field_ideal.py",
line 194:
> sage: NumberField(x^2 + 1, 'a').ideal(7).__hash__()
> Expected:
> -2147483619
> Got:
> -67108835
> }}}
>
> I'm inclined to merge this into 4.6.alpha3. We can open a new ticket
for the new error, unless it indicates a serious problem. I'd like to
release 4.6.alpha3 in a day or so, so please let me know as soon as
possible.
Personally, I think it would be best to fix it first. Otherwise it strikes
me of this comment
http://trac.sagemath.org/sage_trac/ticket/6456#comment:67
by Peter Jeremy.
----
'''I am very concerned at this "release it now, we'll make it work later"
mentality.'''
----
If it is on the strict understanding it does not get into a release until
fixed, then I'm OK with it. That is the purpose of alphas. But I thought
the intension was to have a feature freeze after this alpha. Merging this
could be dangerous thing to do.
The ticket has been open two years - I would have thought those working on
it would have had time to checked it!
Dave
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4000#comment:125>
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.
