auto pep8 format with black

[shimwell]

I ran the black formatter on the code and looks like we have a few places where the formatting could be changed to comply with pep8.

the main changes are white space and indentation

This was done with the default settings for black

pip install black
black .

[psauvan]

what happens when we are not agree with the PEP8 standard?

How can I preserve the matrix structure, since I think it is more readable than the PEP8 format

[shimwell]

Thanks both. I've added the fmt: off to protect this matrix function.

[psauvan]

Same question.
From me my format is more reablable than PEP8 standard.

[AlvaroCubi]

You can protect the code snippets where you want to preserve structure by using the comment # fmt: off

# fmt: off
np.array(
    [
        [1, 0, 0, 0],
        [0, -1, 0, 0],
        [0, 0, 1, 0],
        [0, 0, 0, -1],
    ]
)
# fmt: on

[shimwell]

I think the main point of black is not really if we like the layout or not. It is to have a single layout that we all use. As our own preferences are sometimes subjective and there can be endless disagreement between layouts preference. Black is uncompromising and not very flexible to remove the time spent by developers who all have a different preference on a layout. Sometimes the formatting changes are hints that the code could be done in a different way from the start. I personally don't like the way black formats things but I love it that I never have to think about formatting again