Clean code

[joamatab]

• simpler code with fstrings
• run black

[atait]

1

I prefer this pattern when there is hierarchy equivalence. It makes more sense

if thing:
    return 1
else:
    return 2

The fall through pattern makes more sense when the branches are not logically balanced

if anomaly:
    return None
return 2

Does black have a setting to ignore these?

2

In cases where there are one-line comments, style A

some = code  # this does nothing

style B

# this does nothing
some = code

style C

some = (
    code
)  # this does nothing

The point of style A is for that comment to pertain to that line only, rather than any following lines, which happens in style B. I think style C is the worst. Is there is a way for black to ignore comments when calculating line lengths? Otherwise, can you change these instances to style B?

[joamatab]

sure, let me send them in separate PRs

[atait]

Could you send them as new commits on this PR? Makes things cleaner