Add pandas-path integration
- Adds
.cloudaccessor topd.Seriesandpd.Indexvia a pandas-path custom accessor - Adds simple tests for the integration
- Adds docs for the integration
- Updates changelog
🚀 Deployed on https://deploy-preview-135--gallant-agnesi-5f7bb3.netlify.app
Codecov Report
Merging #135 (fe6f09f) into master (80b596f) will decrease coverage by
0.1%. The diff coverage is66.6%.
@@ Coverage Diff @@
## master #135 +/- ##
========================================
- Coverage 93.5% 93.4% -0.2%
========================================
Files 21 22 +1
Lines 1119 1125 +6
========================================
+ Hits 1047 1051 +4
- Misses 72 74 +2
| Impacted Files | Coverage Δ | |
|---|---|---|
| cloudpathlib/pandas/cloud/__init__.py | 66.6% <66.6%> (ø) |
@jayqi Let me know if you have thoughts on putting this functionality in cloudpathlib. Now that it is implemented, I could also see this making sense in the pandas-path package rather than here. I don't feel strongly either way.
Yeah, I guess it could go either way:
pip install cloudpathlib[s3]
pip install pandas-path
from cloudpathlib.pandas import cloud
pip install pandas-path[s3]
from pandas_path import cloud
Now that I've seen it, I guess the interface looks cleaner to have this live in pandas-path.
@pjbull the more I've thought about this one, the more that I think having this as an extension within pandas_path makes more sense than including it in cloudpathlib.