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

Complex Numbers: Add benchmarks #2506

Open
andrerfcsantos opened this issue Oct 2, 2022 · 2 comments
Open

Complex Numbers: Add benchmarks #2506

andrerfcsantos opened this issue Oct 2, 2022 · 2 comments
Assignees
Labels
x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:size/small Small amount of work x:status/claimed Someone is working on this issue x:type/content Work on content (e.g. exercises, concepts)

Comments

@andrerfcsantos
Copy link
Member

andrerfcsantos commented Oct 2, 2022

On one of my recent mentoring sessions, I noticed the exercise complex-numbers doesn't have any benchmarks.

Benchmarks are not as critical in this exercise as it is in others, but some functions can be optimized, and would be nice to have benchmarks to show students what those improvements can mean performance-wise.

Contributing to this issue

Anyone is welcome to contribute to this issue. If you are interested in working on this, just post a comment below saying you want to work on this.

This issue requires you to write one or more benchmark functions. Ideally each function the student is required to write for the exercise should have a corresponding benchmark function.

Here are some links to help you on this task:

@andrerfcsantos andrerfcsantos added x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:type/content Work on content (e.g. exercises, concepts) x:size/small Small amount of work labels Oct 2, 2022
@W8CYE
Copy link
Contributor

W8CYE commented Oct 2, 2022

@andrerfcsantos I'll take this one.

@andrerfcsantos andrerfcsantos added the x:status/claimed Someone is working on this issue label Oct 2, 2022
@andrerfcsantos
Copy link
Member Author

@W8CYE All yours. Thanks for the interest in tackling this 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/improve Improve existing functionality/content x:knowledge/none No existing Exercism knowledge required x:module/practice-exercise Work on Practice Exercises x:size/small Small amount of work x:status/claimed Someone is working on this issue x:type/content Work on content (e.g. exercises, concepts)
Projects
None yet
Development

No branches or pull requests

2 participants