#15919: Notation S[i] for i-th element of enumerated set S conflicts with
notation
R[x] for polynomial ring
-------------------------------------+-------------------------------------
Reporter: darij | Owner:
Type: defect | Status: needs_review
Priority: major | Milestone: sage-6.2
Component: algebra | Resolution:
Keywords: notation, | Merged in:
algebra, polynomial | Reviewers:
Authors: | Work issues:
Report Upstream: N/A | Commit:
Branch: | 79d469861431ac8d6be5ad4edeace1c6ac8a62c8
u/tscrim/ticket/15919 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by nthiery):
Replying to [comment:17 tscrim]:
> (Much) More of me says this change to `InstanceTester.some_elements()`
is significantly better (and might even be faster for things that are hard
to repeatably iterate over like crystals) than loosing the possibility
that (over enough repeated tests) we get "most" elements of a very large
finite set (and without calculating `__len__()`, which also might be
expensive or error catching). So I'm saying let's make this change, the
question is where. I'd be happy doing it right here.
Ok, glad to see we are on the same line. I have created #16244 for
just this piece. I'll push my little change there later.
> I'd say the problem with `ZZ` is that it is not in the correct category,
which is now #16239. I've also made some very minor doctweaks, so I'm
pos_rev with the current state (up to where to make the change above).
Ok, I am happy with your change. So I guess this can be positive reviewed.
There remains to update the description of this ticket to better reflect
what has actually been done, and refer to the other ticket for the actual
solution of the original issue.
--
Ticket URL: <http://trac.sagemath.org/ticket/15919#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.
