Skip to content

Commit

Permalink
Cannot specify a generating set when multiplying two-sided ideals.
Browse files Browse the repository at this point in the history
  • Loading branch information
tscrim committed Aug 12, 2022
1 parent 66c079b commit c41575e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions src/sage/rings/noncommutative_ideals.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -398,8 +398,7 @@ class Ideal_nc(Ideal_generic):
NotImplementedError: cannot multiply non-commutative ideals
"""
if isinstance(other, Ideal_nc) and self.ring() == other.ring():
if (self.side() == "left" and other.side() == "right"
or self.side() == other.side() == "twosided"):
if self.side() == "left" and other.side() == "right":
gens = [z for z in (x * y for x in self.gens() for y in other.gens()) if z]
return self.ring().ideal(gens, side='twosided')
raise NotImplementedError("cannot multiply non-commutative ideals")
Expand Down

0 comments on commit c41575e

Please sign in to comment.