Skip to content

Commit

Permalink
Fix style in comments and tests
Browse files Browse the repository at this point in the history
  • Loading branch information
syndrakon authored and syndrakon committed Feb 1, 2024
1 parent 6145cff commit b47a604
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/sage/algebras/quatalg/quaternion_algebra.py
Original file line number Diff line number Diff line change
Expand Up @@ -2394,7 +2394,8 @@ def _richcmp_(self, right, op):
sage: I != I # indirect doctest
False

Check warning on line 2396 in src/sage/algebras/quatalg/quaternion_algebra.py

View check run for this annotation

Codecov / codecov/patch

src/sage/algebras/quatalg/quaternion_algebra.py#L2396

Added line #L2396 was not covered by tests
TESTS:
TESTS::
sage: B = QuaternionAlgebra(QQ,-1,-11)
sage: i,j,k = B.gens()
sage: I = B.ideal([1,i,j,i*j])
Expand All @@ -2419,7 +2420,6 @@ def _richcmp_(self, right, op):
True
sage: O >= O
True
"""
return self.free_module().__richcmp__(right.free_module(), op)

Expand Down
4 changes: 3 additions & 1 deletion src/sage/modular/quatalg/brandt.py
Original file line number Diff line number Diff line change
Expand Up @@ -1365,7 +1365,9 @@ def right_ideals(self, B=None):
ideals_theta[J_theta] = [J]
verbose("found %s of %s ideals" % (len(ideals), self.dimension()), level=2)
if len(ideals) >= self.dimension():
#order by basis matrix (as ideals were previously ordered) for backward compatibility and deterministic order of the output
# order by basis matrix (as ideals were previously
# ordered) for backward compatibility and
# deterministic order of the output
ideals = tuple(sorted(ideals, key=lambda x: x.basis_matrix()))
self.__right_ideals = ideals
return ideals
Expand Down

0 comments on commit b47a604

Please sign in to comment.