eja: add random_instance() method for algebras.

[sage.d.git] / mjo / eja / TODO

diff --git a/mjo/eja/TODO b/mjo/eja/TODO
index 67f390ba24f06e6d0f419609ddb8654afd0a65bf..f6d7743782613d241c50e05eee665d59278bd4b4 100644 (file)
--- a/mjo/eja/TODO
+++ b/mjo/eja/TODO
@@ -9,9 +9,13 @@
 5. Factor out the unit-norm basis (and operator symmetry) tests once
    all of the algebras pass.
 
-6. Create Element subclasses for the matrix EJAs, and then override
-   their characteristic_polynomial() method to create a new algebra
-   over the rationals (with a non-normalized basis). We can then
-   compute the charpoly quickly by passing the natural representation
-   of the given element into the new algebra and computing its charpoly
-   there. (Relies on the theory to ensure that the charpolys are equal.)
\ No newline at end of file
+6. Refactor the current ungodly fast charpoly hack (relies on the
+   theory to ensure that the charpolys are equal.)
+
+7. If we factor out a "matrix algebra" class, then it would make sense
+   to replace the custom embedding/unembedding functions with static
+   _real_embedding() and _real_unembedding() methods.
+
+8. Implement random_instance() for the main EJA class.
+
+9. Implement random_instance() for the subalgebra class.