Skip to content

Conversation

@metasim
Copy link
Member

@metasim metasim commented Jul 26, 2019

No description provided.

metasim added 5 commits July 25, 2019 10:09
* develop:
  Docs on raster write: samples, parquet
  Move numpy-pandas doc to pymd format; initial content
  Small docs tweaks lingering from PR 187
  raster reader can take a Pandas DataFrame or geopandas GeoDF
  change link on line 61 to go to nodata-handling.md
  fix tile stats and histogram functions; promote headings up 1 level; remove rf_cell_types
  convert function ref code chunks to running code
@metasim metasim changed the title [WIP] Writing GeoTIFFs Writing GeoTIFFs Jul 29, 2019
@metasim metasim requested a review from vpipkt July 29, 2019 15:02
Copy link
Member

@vpipkt vpipkt left a comment

Choose a reason for hiding this comment

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

A few minor docs comments and python api comments for now. There is a lot here. I am going to try it out!

@vpipkt
Copy link
Member

vpipkt commented Jul 30, 2019

Excited that my initial test is looking very good!

@vpipkt
Copy link
Member

vpipkt commented Jul 30, 2019

One interesting thing.

I read and wrote to gtiff a single MODIS granule (single channel). In the resulting gdalinfo the pixel size is (460.720950959172399,-460.720950959172399)

But the original granule's pixel size is slightly larger: (463.312716527916905,-463.312716527916677)

See the movie attached, you can kind of see an artifact that may be the tile boundaries.

Happy to share original and written file.

ScreenRecordModis.mov.zip

The tile dimensions are pretty much normal for this kind of file, so pretty irregular. The most common one is not even square.

+------------------+-----+
|rf_dimensions(B01)|count|
+------------------+-----+
|         [256, 96]|    6|
|        [257, 257]|   18|
|         [257, 96]|    3|
|          [97, 96]|    1|
|         [97, 257]|    6|
|        [256, 257]|   36|
|        [257, 256]|    9|
|        [256, 256]|   18|
|         [97, 256]|    3|
+------------------+-----+

@metasim metasim merged commit 86b64ed into locationtech:develop Aug 1, 2019
@metasim metasim deleted the docs/raster-writing-1 branch August 1, 2019 12:59
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