cloudpathlib icon indicating copy to clipboard operation
cloudpathlib copied to clipboard

Add pandas-path integration

Open pjbull opened this issue 5 years ago • 5 comments

  • Adds .cloud accessor to pd.Series and pd.Index via a pandas-path custom accessor
  • Adds simple tests for the integration
  • Adds docs for the integration
  • Updates changelog

pjbull avatar Mar 15 '21 06:03 pjbull

🚀 Deployed on https://deploy-preview-135--gallant-agnesi-5f7bb3.netlify.app

github-actions[bot] avatar Mar 15 '21 07:03 github-actions[bot]

Codecov Report

Merging #135 (fe6f09f) into master (80b596f) will decrease coverage by 0.1%. The diff coverage is 66.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%> (ø)

codecov[bot] avatar Mar 15 '21 07:03 codecov[bot]

@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.

pjbull avatar Mar 20 '21 05:03 pjbull

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.

jayqi avatar Mar 20 '21 16:03 jayqi

@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.

jayqi avatar Feb 05 '22 19:02 jayqi