xarray-contrib / xvec

Vector data cubes for Xarray
https://xvec.readthedocs.io
MIT License
93 stars 9 forks source link

ENH: support wide form GeoDataFrames export #30

Closed martinfleis closed 1 year ago

martinfleis commented 1 year ago

Adding a direct support of wide from GeoDataFrames as an export format. May behave strangely in some cases but seems to work well in most common cases when you'd want this.

This branch is built on top of #29 for now, will rebase once that is merged. See the IO notebook for examples.

codecov[bot] commented 1 year ago

Codecov Report

Base: 98.97% // Head: 98.99% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (5c7e1b5) compared to base (93d45a3). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #30 +/- ## ========================================== + Coverage 98.97% 98.99% +0.01% ========================================== Files 3 3 Lines 294 298 +4 ========================================== + Hits 291 295 +4 Misses 3 3 ``` | [Impacted Files](https://codecov.io/gh/martinfleis/xvec/pull/30?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martin+Fleischmann) | Coverage Δ | | |---|---|---| | [xvec/accessor.py](https://codecov.io/gh/martinfleis/xvec/pull/30?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martin+Fleischmann#diff-eHZlYy9hY2Nlc3Nvci5weQ==) | `100.00% <100.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martin+Fleischmann). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martin+Fleischmann)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

martinfleis commented 1 year ago

I have fixed the unsticking so we always should end with non-duplicated geometries (and possibly many-level mutliindex columns) but have not updated the tests as I'm hitting https://github.com/geopandas/geopandas/pull/2088.

martinfleis commented 1 year ago

@benbovy maybe this is a last thing prior release?

codecov[bot] commented 1 year ago

Codecov Report

Merging #30 (3f0840d) into main (d2a7dba) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #30      +/-   ##
==========================================
+ Coverage   98.98%   99.00%   +0.01%     
==========================================
  Files           3        3              
  Lines         297      301       +4     
==========================================
+ Hits          294      298       +4     
  Misses          3        3              
Impacted Files Coverage Δ
xvec/accessor.py 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.