> > There are (at least) two bugs here: > > (1) sB is not in CommutativeRings() > > I'll fix this right now. > > (2) PowerSeriesRing should use > > base_ring in CommutativeRings() > > rather than > > isinstance(base_ring, commutative_ring.CommutativeRing): > > Would you be fine writing and handling a patch for this? > OK
I have uploaded a patch on the combinat patch server. It includes a doctest of a power series version of a Newton identity. I didn t replace isinstance(base_ring, commutative_ring.CommutativeRing) by base_ring in CommutativeRings() but rather put an "or" (was afraid to screw up things for other rings) paul --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/sage-combinat-devel?hl=en -~----------~----~----~----~------~----~------~--~---
