search

icecube / flarestack

Unbinned likelihood analysis code for astroparticle physics datasets
https://flarestack.readthedocs.io/en/latest/?badge=latest
MIT License
8 stars 7 forks source link

Compliance with black v23 #268

Closed mlincett closed 1 year ago

mlincett commented 1 year ago

Addressing #251, but excluding flarestack/analyses.

Aside from the removal of empty lines, there are a few occurrences of for (a, b) in changed for a, b in . I would not worry too much about git blame for these.

robertdstein commented 1 year ago

What's the motivation to exclude flarestack/analyses? The formatting is harmless/aesthetic only, so I see no problem changing analysis code.

mlincett commented 1 year ago

What's the motivation to exclude flarestack/analyses? The formatting is harmless/aesthetic only, so I see no problem changing analysis code.

I was just thinking of linting as something that helps with active code and not much useful on frozen parts, but I am nothing really against running on the entire repo. I am gonna proceed then.