Skip to content

Conversation

MarcoGorelli
Copy link
Collaborator

closes #22983

@MarcoGorelli MarcoGorelli changed the title fix: Method was inconsistent with Python and pandas when string contained leading '+' fix: Method str.zfill was inconsistent with Python and pandas when string contained leading '+' May 28, 2025
@github-actions github-actions bot added fix Bug fix python Related to Python Polars rust Related to Rust Polars labels May 28, 2025
Copy link

codecov bot commented May 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.30%. Comparing base (35f6e9f) to head (7636953).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #22985      +/-   ##
==========================================
- Coverage   80.33%   80.30%   -0.03%     
==========================================
  Files        1682     1682              
  Lines      223120   223140      +20     
  Branches     2804     2804              
==========================================
- Hits       179251   179203      -48     
- Misses      43201    43268      +67     
- Partials      668      669       +1     

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

@MarcoGorelli MarcoGorelli marked this pull request as ready for review May 28, 2025 11:15
@ritchie46 ritchie46 merged commit 42c9a8b into pola-rs:main May 28, 2025
28 checks passed
rapids-bot bot pushed a commit to rapidsai/cudf that referenced this pull request Aug 5, 2025
Closes #19035
Closes #16480

I believe this needs pola-rs/polars#22985 to pass the one remaining failing test and the column overload described here #19035 (comment)

Authors:
  - https://github.com/brandon-b-miller
  - Matthew Roeschke (https://github.com/mroeschke)

Approvers:
  - Matthew Roeschke (https://github.com/mroeschke)

URL: #19081
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Bug fix python Related to Python Polars rust Related to Rust Polars
Projects
None yet
Development

Successfully merging this pull request may close these issues.

zfill with leading + differs from Python stdlib and from pandas
2 participants