andydude Wrote:Also, your usage of __call__ goes against Sage usage. For example, PowerSeries implements coefficients through __getitem__ or f[n] while you implement coefficients through __call__ or f(n), which should probably use __getitem__ instead for this. Also, in PowerSeries, __call__ is associated with what you use .compose for, so __call__ should use this in PowerSeriesI as well. If it is OK, I will try and change these in the git repository...
No, I got the same idea and just changed it. See attachment here
But compose is not expressed via __call__. You dont write f(g), either you write f(g(x)) or you write fog.
Accordingly I also changed the compose to o. So that you can now write f.o(g)