search

alan-turing-institute / deepsensor

A Python package for tackling diverse environmental prediction tasks with NPs.
https://alan-turing-institute.github.io/deepsensor/
MIT License
94 stars 16 forks source link

Fix typo in docs #92

Closed nilsleh closed 1 year ago

nilsleh commented 1 year ago

Just adding a missing character in the otherwise super insightful new documentation user guide!

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

nilsleh commented 1 year ago

I tried but did not succeed in only changing the typo without changing some of the notebook metadata, maybe you can give me a pointer @kallewesterling ?

tom-andersson commented 1 year ago

Hey @nilsleh, thank you for catching this typo!

We're still working on clear documentation contribution guidelines and not really soliciting contributions just yet. For example, the notebooks may require some special care to avoid merge conflicts due to the metadata changing. In the meantime we have issue https://github.com/tom-andersson/deepsensor/issues/87 to collect feedback. Will close this in this instance, but I'm currently working on the finishing touches to the first draft of our docs and I'll incorporate your correction. Thanks again and keep an eye on the DeepSensor Slack for any updates.

P.S. In future for something small like a one-character change, it might be easier to just mention in https://github.com/tom-andersson/deepsensor/issues/87 for someone to pick up when pushing more extensive changes.

cc @kallewesterling in case of further comment