Skip to content

TST: Add more tests for rasterio engine - #293

Merged
snowman2 merged 3 commits into
corteva:masterfrom
snowman2:test
Apr 13, 2021
Merged

TST: Add more tests for rasterio engine#293
snowman2 merged 3 commits into
corteva:masterfrom
snowman2:test

Conversation

@snowman2

@snowman2snowman2 commented Apr 13, 2021

Copy link
Copy Markdown
Member

Continuation of #281

In the process of doing this:

  1. I noticed xarray.open_rasterio was being tested instead of rioxarray.open_rasterio, so I updated that.
  2. Added fix for missing arguments in the engine.

@codecov

codecovBot commented Apr 13, 2021

Copy link
Copy Markdown

Codecov Report

Merging #293 (e5e10cf) into master (497c718) will increase coverage by 1.30%.
The diff coverage is 0.00%.

❗ Current head e5e10cf differs from pull request most recent head 06baf6f. Consider uploading reports for the commit 06baf6f to get more accurate results
Impacted file tree graph

@@ Coverage Diff @@## master #293 +/- ##
==========================================
+ Coverage 92.61% 93.91% +1.30% 
==========================================
Files 13 13 Lines 1395 1398 +3 ==========================================
+ Hits 1292 1313 +21 + Misses 103 85 -18 
Impacted FilesCoverage Δ
rioxarray/xarray_plugin.py0.00% <0.00%> (ø)
rioxarray/raster_array.py97.61% <0.00%> (-0.69%)⬇️
rioxarray/_io.py92.52% <0.00%> (+6.37%)⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 497c718...06baf6f. Read the comment docs.

@snowman2
snowman2 merged commit fe35810 into corteva:masterApr 13, 2021
@snowman2
snowman2 deleted the test branch April 14, 2021 18:13
Sign up for freeto 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

@snowman2@justingruca