X-Git-Url: http://gitweb.michael.orlitzky.com/?a=blobdiff_plain;f=mjo%2Feja%2FTODO;h=b27f2f12875e317fd2b088a74d7395c84ab8e454;hb=c089560955d306b4c2408b222012747c8fe3bddc;hp=e8a2a8d4fc8988b1ad6f744a69daba9cf0582397;hpb=3e46389a46db107db3fe36ace6fe5f2c2b52f815;p=sage.d.git diff --git a/mjo/eja/TODO b/mjo/eja/TODO index e8a2a8d..b27f2f1 100644 --- a/mjo/eja/TODO +++ b/mjo/eja/TODO @@ -11,3 +11,22 @@ DirectSumEJA. 6. Switch to QQ in *all* algebras for _charpoly_coefficients(). + This only works when we know that the basis can be rationalized... + which is the case at least for the concrete EJAs we provide, + but not in general. + +7. Pass already_echelonized (default: False) and echelon_basis + (default: None) into the subalgebra constructor. The value of + already_echelonized can be passed to V.span_of_basis() to save + some time, and usinf e.g. FreeModule_submodule_with_basis_field + we may somehow be able to pass the echelon basis straight in to + save time. + + This may require supporting "basis" as a list of basis vectors + (as opposed to superalgebra elements) in the subalgebra constructor. + +8. Implement random_instance() for general algebras as random_eja(). + Copy/paste the "general" construction into the other classes that + can use it. The general construction can be something like "call + random_instance() on something that inherits me and return the + result."