Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[EXPRESSIONS] str.repeat(n) #2074

Closed
Tracked by #1037
colin-ho opened this issue Apr 2, 2024 · 5 comments · Fixed by #2198
Closed
Tracked by #1037

[EXPRESSIONS] str.repeat(n) #2074

colin-ho opened this issue Apr 2, 2024 · 5 comments · Fixed by #2198
Assignees
Labels
expression A new expression good first issue Good for newcomers

Comments

@colin-ho
Copy link
Contributor

colin-ho commented Apr 2, 2024

Repeats a string n times.
https://ibis-project.org/reference/expression-strings#ibis.expr.types.strings.StringValue.repeat

See #2020 for reference on how to add a string expression

@colin-ho colin-ho mentioned this issue Apr 2, 2024
36 tasks
@colin-ho colin-ho added good first issue Good for newcomers expression A new expression labels Apr 2, 2024
@aeroaks
Copy link

aeroaks commented Apr 3, 2024

I would like to pick it up.

@colin-ho
Copy link
Contributor Author

colin-ho commented Apr 3, 2024

Great! Assigning this to you, thanks @aeroaks !!

@samster25
Copy link
Member

Hi @aeroaks! Wondering if you had any updates for this issue?

@aeroaks
Copy link

aeroaks commented Apr 10, 2024

Hi,
I did spend some time on it. This is significantly unknown codebase, that too in Rust, so took some time figuring out. I have done the definitions and major code referring to the str.extract PR. I am now at the core of the functionality and trying to get it built and tested.

@murex971
Copy link
Contributor

Hey @aeroaks are you still working on this issue ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expression A new expression good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants