Nbqa black

[MarcoGorelli]

closes #4095

~ 5 notebooks per commit

it's somewhat a matter of choice when/where to turn fmt: off and fmt: on, but I've tried to make sensible choices

---

a note on the fixup in docs/source/notebooks/SMC2_gaussians.ipynb: that notebook originally had

az.plot_ess(az_trace, coords={"X_dim_0": slice(0, 5)}); # only the first 6 dimensions

Because there's a comment after the trailing semi-colon, both nbqa black and black_nbconvert would transform it to

az.plot_ess(az_trace, coords={"X_dim_0": slice(0, 5)})
# only the first 6 dimensions

Normally, nbqa is able to preserve these trailing semicolons, but here the extra comment gets in our way. We'd like to address this eventually (nbQA-dev/nbQA#282) but it's not-so-straightfoward to do correctly, so for now we're treating it as a known limitation which can be avoided by putting the comment on a separate line.

Apart from this and the occasional fmt: off and fmt: on I've added, the rest is just the output from having applied nbqa black

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[codecov]

Codecov Report

Merging #4168 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4168   +/-   ##
=======================================
  Coverage   88.76%   88.76%           
=======================================
  Files          89       89           
  Lines       14083    14083           
=======================================
  Hits        12501    12501           
  Misses       1582     1582           

[AlexAndorra]

All good, thanks @MarcoGorelli ! And all done in one PR 🎉

[MarcoGorelli]

Thank you, sorry for the tediousness of reviewing this kind of change! On the plus side, it's done now 🎉