From: Michael Orlitzky <michael@orlitzky.com>
Date: Sun, 22 Nov 2020 15:50:45 +0000 (-0500)
Subject: eja: remove a completed TODO and update an in-progress one.
X-Git-Url: https://gitweb.michael.orlitzky.com/?a=commitdiff_plain;h=2a3ec0787b28bc35b3624594d921a995c9425d3a;p=sage.d.git

eja: remove a completed TODO and update an in-progress one.
---

diff --git a/mjo/eja/TODO b/mjo/eja/TODO
index f248e11..87a162c 100644
--- a/mjo/eja/TODO
+++ b/mjo/eja/TODO
@@ -7,8 +7,7 @@
 4. Factor out the unit-norm basis (and operator symmetry) tests once
    all of the algebras pass.
 
-5. Override inner_product(), _max_test_case_size(), et cetera in
-   DirectSumEJA.
+5. Override random_instance(), one(), et cetera in DirectSumEJA.
 
 6. Switch to QQ in *all* algebras for _charpoly_coefficients().
    This only works when we know that the basis can be rationalized...
@@ -30,6 +29,3 @@
    can use it. The general construction can be something like "call
    random_instance() on something that inherits me and return the
    result."
-
-9. Pre-cache the one() method for concrete algebras, and test the general
-   method by clearing the cache.