I have finished making our btquotients code use the distributions in pollack-stevens. The speed seems to be comparable (currently I have only tried Dist_vector, which is the one that would really compare to our implementation anyway), so I think we can safely remove ocmodule.py (although I'm going to leave it there for some days, just in case we change our mind!).
What I'd like to ask is that you test (sage -t) pautomorphicform.py before committing changes, since right now doctests are passing. This way we will be checking that all the functions respect the non-default adjuster parameter, as well as working with matrices with coefficients other than ZZ.
I have finished making our btquotients code use the distributions in pollack-stevens. The speed seems to be comparable (currently I have only tried Dist_vector, which is the one that would really compare to our implementation anyway), so I think we can safely remove ocmodule.py (although I'm going to leave it there for some days, just in case we change our mind!).
What I'd like to ask is that you test (sage -t) pautomorphicform.py before committing changes, since right now doctests are passing. This way we will be checking that all the functions respect the non-default adjuster parameter, as well as working with matrices with coefficients other than ZZ.