Skip to content
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

Fix typos #1963

Merged
merged 4 commits into from
Oct 18, 2023
Merged

Fix typos #1963

merged 4 commits into from
Oct 18, 2023

Conversation

giswqs
Copy link
Contributor

@giswqs giswqs commented Oct 18, 2023

Overview

Fix many typos in the text detected by codespell.

Checklist

  • Added needs-backport label if PR is bug fix that applies to previous minor release
  • Ran scripts/format_code and committed any changes
  • Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

Notes

Optional. Ancillary topics, caveats, alternative strategies that didn't work out, anything else.

Testing Instructions

  • How to test this PR
  • Prefer bulleted description
  • Start after checking out this branch
  • Include any setup required, such as rebuilding the Docker image.
  • Include test case, and expected output if not captured by automated tests.

Closes #XXX

Copy link
Collaborator

@AdeelH AdeelH left a comment

Choose a reason for hiding this comment

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

Thank you for the valuable PR!

We require external contributors to sign a CLA before merging their changes. The CLA and instructions on how to submit it are provided here: https://docs.rastervision.io/en/stable/CONTRIBUTING.html.

Let me know once you have done so and thank you once again for the PR!

@AdeelH AdeelH added the needs-backport This PR needs to be backported to release branches label Oct 18, 2023
@giswqs
Copy link
Contributor Author

giswqs commented Oct 18, 2023

CLA sent. Please check. Thanks.

@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #1963 (b7d6865) into master (a130639) will decrease coverage by 0.27%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1963      +/-   ##
==========================================
- Coverage   82.47%   82.21%   -0.27%     
==========================================
  Files         190      190              
  Lines        9337     9337              
==========================================
- Hits         7701     7676      -25     
- Misses       1636     1661      +25     
Files Coverage Δ
rastervision_core/rastervision/core/box.py 100.00% <ø> (ø)
...vision_core/rastervision/core/data/class_config.py 92.18% <ø> (ø)
...on/core/data/label/semantic_segmentation_labels.py 91.53% <ø> (ø)
...data/label_source/object_detection_label_source.py 83.63% <ø> (ø)
.../rastervision/core/data/label_store/label_store.py 100.00% <ø> (ø)
...rvision/core/data/raster_source/rasterio_source.py 81.52% <ø> (ø)
...tervision/core/data/raster_source/xarray_source.py 92.15% <ø> (ø)
...ision_core/rastervision/core/data/utils/factory.py 81.94% <ø> (ø)
...ision_core/rastervision/core/data/utils/geojson.py 97.51% <100.00%> (ø)
...vision_core/rastervision/core/data/utils/raster.py 97.05% <ø> (ø)
... and 15 more

... and 2 files with indirect coverage changes

Copy link
Collaborator

@AdeelH AdeelH left a comment

Choose a reason for hiding this comment

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

Thank you!

@AdeelH AdeelH merged commit e3abaf4 into azavea:master Oct 18, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-backport This PR needs to be backported to release branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants