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

Misc. minor fixes #1984

Merged
merged 6 commits into from Nov 8, 2023
Merged

Misc. minor fixes #1984

merged 6 commits into from Nov 8, 2023

Conversation

AdeelH
Copy link
Collaborator

@AdeelH AdeelH commented Nov 7, 2023

Overview

This PR makes multiple minor code quality improvements. See commits for details.

Checklist

  • Added unit tests, if applicable
  • Updated documentation, if applicable
  • Added needs-backport label if the change should be back-ported to the previous release
  • PR has a name that won't get you publicly shamed for vagueness

Notes

N/A

Testing Instructions

N/A

Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #1984 (93a948b) into master (cd102da) will increase coverage by 0.02%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #1984      +/-   ##
==========================================
+ Coverage   86.14%   86.17%   +0.02%     
==========================================
  Files         195      195              
  Lines        9581     9594      +13     
==========================================
+ Hits         8254     8268      +14     
+ Misses       1327     1326       -1     
Files Coverage Δ
...vision_core/rastervision/core/analyzer/analyzer.py 100.00% <ø> (ø)
...ervision_core/rastervision/core/backend/backend.py 100.00% <ø> (ø)
...ision/core/data/crs_transformer/crs_transformer.py 100.00% <ø> (ø)
...vision_core/rastervision/core/data/label/labels.py 100.00% <ø> (ø)
...on/core/data/label/semantic_segmentation_labels.py 91.53% <ø> (ø)
...astervision/core/data/label_source/label_source.py 78.57% <100.00%> (+3.57%) ⬆️
.../rastervision/core/data/label_store/label_store.py 100.00% <100.00%> (ø)
...a/label_store/semantic_segmentation_label_store.py 94.41% <100.00%> (ø)
...tervision/core/data/raster_source/raster_source.py 91.30% <100.00%> (+0.25%) ⬆️
...core/data/raster_transformer/raster_transformer.py 100.00% <ø> (ø)
... and 14 more

... and 1 file with indirect coverage changes

@AdeelH AdeelH marked this pull request as ready for review November 8, 2023 14:18
@AdeelH AdeelH merged commit f11e17a into azavea:master Nov 8, 2023
2 checks passed
@AdeelH AdeelH deleted the misc branch November 8, 2023 14:19
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.

None yet

1 participant