[PyOpenSci] Documentation and Notebook Adjustments, manage SBCK

[Zeitsperre]

Pull Request Checklist:

• This PR addresses an already opened issue (for bug fixes / features)
  • This PR fixes Can't create xclim environment : Failed to build sbck #1332
  • This pr addresses comments raised in [PyOpenSci] Reviewer #2 comments #1335
• Tests for the changes have been added (for bug fixes / features)
  • (If applicable) Documentation has been added / updated (for bug fixes / features)
• CHANGES.rst has been updated (with summary of main changes)
  • Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?

• Grammar and spelling mistakes within the notebooks have been corrected reduced.
• SBCK has been pulled from environment.yml for the time being until a proper package is created.
• The Contributing guide now lists three separate subsections to help users understand the gains from optional dependencies

Does this PR introduce a breaking change?

Yes, but no¹. Changes to dependencies have been handled in such a way that default settings are unaffected. Development and CI-based behaviour is affected: clisops and flox are now in the environment.yml.

Other information:

yrobink/SBCK-python#5

Footnotes

1. https://i.kym-cdn.com/entries/icons/original/000/028/596/dsmGaKWMeHXe9QuJtq_ys30PNfTGnMsRuHuo_MUzGCg.jpg ↩

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[juliettelavoie]

I removed seaborn. As I understand it, the point of this section is to show that the labels are integrated automatically on the plot from the attributes created by xclim. I thought this was illustrated well enough by matplotlib. I didn't see the added value of having seaborn also.

[juliettelavoie]

Re: plt.draw(). I left them there because I assumed there was a reason they were needed (like to show up on RtD or something), but I actually have no idea.

[Zeitsperre]

@juliettelavoie I was getting odd messages if plt.show() wasn't explicitly written. I saw that plt.draw() did what I wanted, so I went with that. This has been removed.