Skip to content

Correct minor tutorial typos #149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Nov 5, 2022
Merged

Correct minor tutorial typos #149

merged 6 commits into from
Nov 5, 2022

Conversation

loganthomas
Copy link
Contributor

I recently watched Anderson Banihirwe's "Xarray Tutorial | xarray fundamentals" on YouTube (here). This PR attempts to correct a few very minor typos that I found while following along in the notebooks.

While it's not a typo fix, I also added a misc section to the .gitignore file to include .DS_Store and .vscode/ so that those will not get pushed accidentally to the repo.

Happy to change anything that might be problematic.

Thank you for the great resources for Xarray. I'm really enjoying the tutorials so far!

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@loganthomas
Copy link
Contributor Author

Updated overview/xarray-in-45-min.ipynb to solve failing check in 8ef9c4a

Copy link
Contributor

@dcherian dcherian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! DataArray is the proper term so I added capital letters to make it clear

@dcherian dcherian enabled auto-merge (squash) November 5, 2022 00:37
@dcherian dcherian merged commit 461423c into xarray-contrib:main Nov 5, 2022
dcherian added a commit to dcherian/xarray-tutorial that referenced this pull request Nov 5, 2022
* main:
  Correct minor tutorial typos (xarray-contrib#149)
  Update backends.md (xarray-contrib#148)
  [pre-commit.ci] pre-commit autoupdate (xarray-contrib#147)
  [pre-commit.ci] pre-commit autoupdate (xarray-contrib#143)
  remove user survery banner (xarray-contrib#145)
  fix typo (xarray-contrib#146)
  Remote data access tutorial for CMIP6 Zarr data (xarray-contrib#132)
  Update 45 minute overview for OceanHackWeek 2022 (xarray-contrib#140)
@loganthomas loganthomas deleted the fix/tutorial-typos branch March 2, 2023 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants