Lots of ruff fixes#44
Conversation
for more information, see https://pre-commit.ci
|
|
||
| existing_gens = existing_gens.rename( | ||
| columns={existing_gens.columns.values[0]: "Generator"} | ||
| columns={existing_gens.columns.to_numpy[0]: "Generator"} |
There was a problem hiding this comment.
.. fyi, .to_numpy is a method.. so should be .to_numpy()[0] (this is a known unsafe auto-fix)
|
To discuss - but I think that a) we shouldn't be using |
There are so many useful linters in Ruff that I think the right way forward is to use |
This is the first round of clean-ups by running
ruff --select=ALL(with a few classes of warnings disabled for now).None of these fixes have highlighted up any bugs, but they do improve readability in many instances.