#14054: Cythoned UniqueRepresentation
---------------------------------------------------------+------------------
Reporter: SimonKing | Owner: tbd
Type: enhancement | Status:
needs_review
Priority: major | Milestone:
sage-5.8
Component: performance | Resolution:
Keywords: cython UniqueRepresentation | Work issues:
Report Upstream: N/A | Reviewers:
Travis Scrimshaw
Authors: Simon King | Merged in:
Dependencies: #14017, #6495, #14182, #14040, #14011 | Stopgaps:
---------------------------------------------------------+------------------
Comment (by SimonKing):
Replying to [comment:77 tscrim]:
> Would it be better to give them an (somewhat unnatural) ordering or to
change the doctest?
>
> Actually...why not just make it `#random` with a note saying the output
order can be random since there is no ordering?
I would be fine with making it random, ''and'' test that the length of the
output (or another easy invariant) is correct.
Is there a mathematically meaningful ordering? From the code, I guess that
rigged configuration elements rely on list of tableaus, and if the
tableaus can be ordered then one might have some kind of lexicographical
ordering, perhaps. But this should be on a different ticket (and perhaps
would not make sense at all), so that I would prefer to change the test.
Jeroen, what is your opinion on that matter?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14054#comment:79>
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.
