Skip to content

feat: add nw.Int128, nw.UInt128#1570

Merged
MarcoGorelli merged 4 commits into
narwhals-dev:mainfrom
MarcoGorelli:huge-ints
Dec 12, 2024
Merged

feat: add nw.Int128, nw.UInt128#1570
MarcoGorelli merged 4 commits into
narwhals-dev:mainfrom
MarcoGorelli:huge-ints

Conversation

@MarcoGorelli
Copy link
Copy Markdown
Member

closes #1569

What type of PR is this? (check all applicable)

  • 💾 Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 🔧 Optimization
  • 📝 Documentation
  • ✅ Test
  • 🐳 Other

Related issues

  • Related issue #<issue number>
  • Closes #<issue number>

Checklist

  • Code follows style guide (ruff)
  • Tests added
  • Documented the changes

If you have comments or can explain your changes, please do so below

@MarcoGorelli MarcoGorelli marked this pull request as ready for review December 12, 2024 15:31
@MarcoGorelli MarcoGorelli merged commit f0395ae into narwhals-dev:main Dec 12, 2024
@MarcoGorelli MarcoGorelli added the enhancement New feature or request label Dec 12, 2024
dangotbanned added a commit that referenced this pull request Sep 18, 2025
@dangotbanned dangotbanned mentioned this pull request Sep 18, 2025
10 tasks
MarcoGorelli pushed a commit that referenced this pull request Oct 3, 2025
* ci(typing): Bump `polars==1.33.1`

Related:
- pola-rs/polars#24334
- pola-rs/polars#24340

```py
narwhals/_polars/utils.py:107:47 - error: "UInt128" is not a known attribute of module "polars" (reportAttributeAccessIssue)
```

* fix: Remove check for non-existent `polars.UInt128`

`polars` only has `pl.Int128`

#1570

* fix: Use a self doc condition w/ the right version

`1.18.0` instead of `1.8.0`

* fix: Partial revert, `pl.UInt128` will soon-exist

#3138 (comment)

* perf: Move `pl.Int128` branch out of `narwhals_to_native_dtype`

* maybe fix coverage?

#3138 (comment)

* ci(typing): Bump `polars==1.34.0`

* fix(typing): Remove ignore fixed by my pr 🥳

pola-rs/polars#24584

* chore: actually integrate `pl.UInt128`

#3138 (comment), #3138 (comment))

* style: Sort version guards in reverse-chrono

* test: Roundtrip coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

enh: add nw.Int128 and nw.UInt128

1 participant