eja: add random_instance() method for algebras.

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

diff --git a/mjo/eja/TODO b/mjo/eja/TODO
index 382fb6e67d6050dd6e8215ad2d371a5be3022057..f6d7743782613d241c50e05eee665d59278bd4b4 100644 (file)
--- a/mjo/eja/TODO
+++ b/mjo/eja/TODO
@@ -6,12 +6,16 @@
 
 4. Implement the octonion simple EJA.
 
-5. Factor out the Jordan axiom and norm 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
+5. Factor out the unit-norm basis (and operator symmetry) tests once
+   all of the algebras pass.
+
+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.