Fix the rest of linting rules

[ddundo]

In #62 I added a Ruff linter, but I ignored errors that couldn't be fixed automatically - see below. We should manually fix them.

ignore = [
    "B006",  # Mutable data structures in argument defaults
    "B007",  # Loop control variable not used within loop body
    "B008",  # Function call `range` in argument defaults
    "B023",  # Function definition does not bind loop variable
    "C405",  # Unnecessary list literal
    "C408",  # Unnecessary `dict()` call
    "C414",  # Unnecessary `list()` call
    "C416",  # Unnecessary list comprehension
    "C901",  # Function too complex
    "E402",  # Module level import not at top of file
    "E501",  # Line too long
    "E712",  # Avoid equality comparisons to `False`
    "E721",  # Use `is` and `is not` for type comparisons, or `isinstance()`
    "E722",  # Using bare `except`
    "F841",  # Local variable assigned to but never used
]