#14287: Split _test_elements_eq
--------------------------------+-------------------------------------------
Reporter: saraedum | Owner: nthiery
Type: enhancement | Status: positive_review
Priority: trivial | Milestone: sage-pending
Component: categories | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: David Roe
Authors: Julian Rueth | Merged in:
Dependencies: #14284 | Stopgaps:
--------------------------------+-------------------------------------------
Comment (by nthiery):
Just a micro comment about the use of this ticket; not its
implementation.
``However, transitivity does not hold in padic rings''
Sounds scary! Well, as long as you know what you are doing :-)
``so they that we can override the transitivity test for padic
implementations''
I am not sure what's your plan for this; just in case, and unless this
becomes too cumbersome, I prefer writing explicitly each time:
{{{
sage: TestSuite(...).run(skip="....transitivity")
}}}
rather than systematically disabling the transitivity test. First,
that's the occasion to warn the user about an atypical behavior. And
to get more people to thing about creative ways of fixing that.
Second, the test might actually pass in certain cases, and it's a
useful information to record.
Cheers,
Nicolas
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14287#comment:11>
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?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
