(Hopefully) Fixed several bugs with respect to the coordinate system used...

There was a bug reported by Sven Möller that caused _action_of_Gamma0 to fail
because this method still used the old ._L list of elements in WeilModuleElement.
However, there werde quite a few bugs regarding the use of the el_index etc. - methods.
This is all due to the fact that can_coords=True has to be used
when the generators and their orders are used for the coordinate functions.
I think this should be changed to using the elementary divisors instead.
But for now, I just fixed the bugs (ugly...)

FREDRIK: Please review this. Especially: There is a line that I changed assuming that "I" should be the usual fourth root of unity contained in the complex upper half-plane. Is this correct? Did I do it correctly??
(line 1219).
As far as I can tell your fix works fine.
But it would of course still be good to write it more cleanly in terms of
elementary divisors or something.
The way we define and use the canonical coordinates is not really
transparent.

Fredrik


On Thu, Jul 17, 2014 at 9:34 PM, Stephan Ehlen <notifications@github.com>
wrote:

> In psage/modules/weil_module.py:
>
> > @@ -1215,7 +1216,7 @@ def _action_of_Gamma0(self,A,filter=None):
> >                  if( d % 4  == 1 ):
> >                      chi = 1
> >                  else:
> > -                    chi=I**power
>
> What is "I" here?
> I assumed that it is the imaginary unit, so it should be equal to z8^2?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fredstro/psage/pull/6/files#r15082209>.
>