2. Profile (and fix?) any remaining slow operations.
-3. Every once in a long while, the test
+3. When we take a Cartesian product involving a trivial algebra, we
+ could easily cache the identity and charpoly coefficients using
+ the nontrivial factor. On the other hand, it's nice that we can
+ test out some alternate code paths...
- sage: set_random_seed()
- sage: x = random_eja().random_element()
- sage: x.is_invertible() == (x.det() != 0)
+4. Add dimension bounds on any tests over AA that compute element
+ subalgebras.
- in eja_element.py returns False.
+5. The rational_algebra() stuff doesn't really belong in classes that
+ don't dervice from RationalBasisEJA or its as-yet-nonexistent
+ element class.