feat: Add support for .shift(n).over('col') for pandas-like DataFrames

[ClaudioSalvatoreArcidiacono]

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

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

Related issues

• Closes enh: support nw.col('a').shift(n).over('b') #1609

Checklist

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

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

See #1609

[MarcoGorelli]

nice, thanks @ClaudioSalvatoreArcidiacono !

I think @FBruzzesi 's work in #1614 might make this easier, perhaps we should wait to get that in, then we can update here?

[FBruzzesi]

Thanks @ClaudioSalvatoreArcidiacono !

I was about to echo what @MarcoGorelli just mentioned! Instead of an ad-hoc change, I think waiting for #1614 would make it easier. Possibly it will be ready by end of today.

[ClaudioSalvatoreArcidiacono]

Definetly! Nice PR @FBruzzesi

[MarcoGorelli]

that PR's merged, if you fancy updating @ClaudioSalvatoreArcidiacono I think we'd love to get this in!

[FBruzzesi]

that PR's merged, if you fancy updating @ClaudioSalvatoreArcidiacono I think we'd love to get this in!

Just a headsup that the PR was shipped with a bug, namely it does not track kwargs in expr composition. You would probably want to wait for #1645 which contains a fix

[FBruzzesi]

Pretty happy with this! I left only a tiny comment before merging!

[FBruzzesi]

Thanks a ton @ClaudioSalvatoreArcidiacono 👌