#21059: Change WithRealizations._an_element_ to use a_realization
------------------------------------+--------------------------
       Reporter:  tscrim            |         Type:  defect
         Status:  new               |     Priority:  major
      Milestone:  sage-7.3          |    Component:  categories
       Keywords:                    |    Merged in:
        Authors:  Travis Scrimshaw  |    Reviewers:
Report Upstream:  N/A               |  Work issues:
         Branch:                    |       Commit:
   Dependencies:                    |     Stopgaps:
------------------------------------+--------------------------
 As noticed on #21054, there is a discrepancy between `one` and
 `_an_element_` for parents that are in the category `*.WithRealizations`,
 in that the former uses `a_realization`, whereas the latter uses
 `realizations()[0]`. While on #21054, this did uncover an error, this
 cases problems with the test suite when a realization does not get created
 before `an_element` gets called (which I have noticed before).

 Thus, I propose to have `_an_element_` use `a_realization()`, which is a
 required implementation and `_an_element_` will also have consistent
 output no matter which basis is created first.

--
Ticket URL: <https://trac.sagemath.org/ticket/21059>
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 https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to