Skip to content

Conversation

dougbrn
Copy link
Collaborator

@dougbrn dougbrn commented Sep 17, 2025

Opening this early to keep track of CI. Major components of this PR have been reviewed as separate PRs into this branch, see #360 and #366

Resolves #355
Resolves #226
Resolves #354
Resolves #370

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

codecov bot commented Sep 17, 2025

Codecov Report

❌ Patch coverage is 97.50000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.78%. Comparing base (0873b32) to head (dd891ae).
⚠️ Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
src/nested_pandas/series/dtype.py 86.95% 3 Missing ⚠️
src/nested_pandas/series/accessor.py 98.59% 1 Missing ⚠️
src/nested_pandas/series/nestedseries.py 93.75% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #363      +/-   ##
==========================================
- Coverage   98.19%   97.78%   -0.41%     
==========================================
  Files          19       19              
  Lines        1880     2035     +155     
==========================================
+ Hits         1846     1990     +144     
- Misses         34       45      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

github-actions bot commented Sep 17, 2025

Before [cb5b36d] After [e6d33da] Ratio Benchmark (Parameter)
31.7±1ms 32.7±0.4ms 1.03 benchmarks.AssignSingleDfToNestedSeries.time_run
135M 139M 1.03 benchmarks.CountNestedBy.peakmem_run
1.33±0.02ms 1.36±0.03ms 1.02 benchmarks.NestedFrameReduce.time_run
1.59±0.03s 1.63±0.05s 1.02 benchmarks.ReadFewColumnsHTTPS.time_run
102M 103M 1.01 benchmarks.NestedFrameAddNested.peakmem_run
11.5±0.3ms 11.6±0.2ms 1.01 benchmarks.NestedFrameAddNested.time_run
107M 107M 1.01 benchmarks.NestedFrameQuery.peakmem_run
105M 106M 1.01 benchmarks.NestedFrameReduce.peakmem_run
177M 179M 1.01 benchmarks.ReadFewColumnsHTTPS.peakmem_run
250M 250M 1 benchmarks.AssignSingleDfToNestedSeries.peakmem_run

Click here to view all benchmarks.

@dougbrn dougbrn mentioned this pull request Sep 22, 2025
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.

Unwrapped list of nested frame columns Renaming for v0.6.0 Allow UDFs to take all columns in reduce() Rename: reduce -> map_rows
1 participant