search

exercism / problem-specifications

Shared metadata for exercism exercises.
MIT License
320 stars 540 forks source link

Improve the description of square root problem #2415

Open Ashvith10 opened 3 months ago

Ashvith10 commented 3 months ago

Continued from exercism/c#967

The test case for the square root problem indicates that it was made only for perfect squares. This is also hinted by the fact that the C track uses uint16_t and not float_t or double_t. However, this important detail is not mentioned in the description. I think that letting the learners know that the argument will be perfect squares would improve the quality of the question and reduce the cognitive load.

github-actions[bot] commented 3 months ago

Hello. Thanks for opening an issue on Exercism 🙂

At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.

This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!

If you're interested in learning more about this auto-responder, please read this blog post.

BNAndras commented 3 months ago

Generally, the test suite is what students should consult for what's being requested. With that out of the way, do you have some proposed text for mentioning the perfect squares in the instructions? It doesn't have to be anything polished, but it'll give us a starting point and we can fine-tune it collaboratively.

IsaacG commented 3 months ago

Please continue any proposals/discussions on the forum.

Ashvith10 commented 3 months ago

@BNAndras and @IsaacG I've created a post on the forum. The idea is that instead of natural radicand, we should let the learners know that we are looking for perfect square randicand.