Although we did adhere to some broad standards in terms of code style, there are still various places where things can be made more homogeneous. Notably, although docstrings (should generally) conform to reST, there's some variation in things like how to distinguish variables in text and the specifics of how whitespace is used. Perhaps it could be worth using an autoformatter (Ruff has worked pretty well for me) over the entire codebase.
On a related note, the current unit tests use a mixture of unittest (self.assertEqual(a, b) and similar inside methods of unittest.TestCase subclasses) and pytest (assert a == b and similar) styles -- probably a good idea to standardize on one style.
Although we did adhere to some broad standards in terms of code style, there are still various places where things can be made more homogeneous. Notably, although docstrings (should generally) conform to reST, there's some variation in things like how to distinguish
variables
in text and the specifics of how whitespace is used. Perhaps it could be worth using an autoformatter (Ruff has worked pretty well for me) over the entire codebase.On a related note, the current unit tests use a mixture of
unittest
(self.assertEqual(a, b)
and similar inside methods ofunittest.TestCase
subclasses) and pytest (assert a == b
and similar) styles -- probably a good idea to standardize on one style.