problem-specifications icon indicating copy to clipboard operation
problem-specifications copied to clipboard

Improve the description of square root problem

Open Ashvith10 opened this issue 1 year ago • 4 comments

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.

Ashvith10 avatar Apr 02 '24 19:04 Ashvith10

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.

github-actions[bot] avatar Apr 02 '24 19:04 github-actions[bot]

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.

BNAndras avatar Apr 03 '24 04:04 BNAndras

Please continue any proposals/discussions on the forum.

IsaacG avatar Apr 03 '24 04:04 IsaacG

@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.

Ashvith10 avatar Apr 03 '24 06:04 Ashvith10

Looks like #2495 resolved this.

tasxatzial avatar Nov 08 '24 11:11 tasxatzial