Explorer.Series.pow does not work when supplied integers as exponent about explorer HOT 9 CLOSED

A bit of context on my rationale to not have this included on the Series trait is that this will be compiled for every datatype in polars.

Doing so for many operations will put a lot of constrain on compile times of polars-core.

The recommended way for a user to apply a numeric operation like pow, sin exp etc. is downcasting the Series to the ChunkedArray and then calling apply

Something like this.

let s = Series::new("foo", [1i32, 2, 3]);

s.i32()?.apply(|v| v.pow(3)).into_series()

Given that Explorer defines bindings to polars, you'd need to pattern match on the dtype and downcast to the appropriate type.

from explorer.

@kimjoaoun that's probably the best in terms of having a record to point to for others contributing. But you can also get me on the EEF or Elixir slacks (same username) or via email at chris at amplified dot ai.

And thanks for taking it on!

from explorer.

Nah, you are being very helpful. Contributions are very welcome! :)

from explorer.

Good catch. Unfortunately this is because polars only takes f64 for the pow method: https://docs.rs/polars/0.18.0/polars/prelude/trait.SeriesTrait.html#method.pow.

I'm sure we could work around it, and at the very least document it!

from explorer.

@kimjoaoun if you could also ping polars folks and ask if they want to support an int version. It could be a good contribution upstream! ❤️

from explorer.

@josevalim I'll ping them!
(se eu estiver sendo chato abrindo várias issues, por favor, me avise)

from explorer.

@josevalim It looks like pow() will be removed, so I think they'll not support an int version. I think we would need to do what ritchie46 suggested and coerce our integer to a float.

from explorer.

Thanks for the clarification @ritchie46! That makes total sense as a preferred path forward.

I'm fighting against a Wednesday deadline with work at the minute but could pick this up after. Otherwise @kimjoaoun if you're keen then I can definitely give some guidance and would be happy to review a PR.

from explorer.

@cigrainger I accept the guidance, I'll start working on that. If I have a question should I contact you via a PR comment?

from explorer.