I213 to raster kwargs

[philippkraft]

Pull request to fix #213. Test is runnning and it works as expected:

grid = Grid.from_raster('data/dem.tif')
dem = grid.read_raster('data/dem.tif')
grid.to_raster(dem, 'data/dem_lzw.tif', compress='lzw')

compression = os.stat('data/dem_lzw.tif').st_size / os.stat('data/dem.tif').st_size  #  --> 36%

[mdbartos]

Looks like CI tests are not yet passing.

[philippkraft]

Ok, I found the reason for this:

The pgrid.to_raster and sgrid.to_raster interfaces are not equal. I have removed my changes to pgrid, should work now.

[mdbartos]

I will update the CI to include newer versions of Python.

[mdbartos]

Hi @philippkraft, sorry for the delay. I am getting ready to pull this in. I updated github actions to remove Python 3.6 and run Python 3.10:

https://github.com/mdbartos/pysheds/blob/76eebd1799f3b70eb96940277c0a984b46d9227e/.github/workflows/python-package.yml#L19

It's not running the test for Python 3.10 in this PR. Would it be possible to modify this line on your branch so that it will run the test for Python 3.10?

Thanks, MDB

[philippkraft]

Hi, I updated my PR's. Feel free to test again.

[mdbartos]

Great. Could you add a quick unit test of the new functionality to test_grid.py?

https://github.com/mdbartos/pysheds/blob/master/tests/test_grid.py

[philippkraft]

Added a test with using compress as a keyword

[mdbartos]

Looks good to me. Will merge now.