Square Root exercise instructions could be clearer

While the README for the Square Root exercise hints that there are efficient iterative solutions available for natural numbers, given that the vast majority of solutions simply use the exponentiation built-in, it might be helpful to clarify that the ** operator is included in the vague phrase “built-ins such as pow()”

Hi @xanni :wave:

Sure. That makes sense. Would you like to submit a PR for that? :smile:

If not, I can get to it this week sometime.

Thanks! I’ve created PR #3812.

1 Like

Is this a Python-specific change or a proposed problem specs change? ** isn’t, AFAIK a universal operator.

Python-specific; as per the existing documentation, I don’t think the math module or the pow() and sum() functions are universal either.

2 Likes

It is in a Python-specific instruction append. Hence the call to make a PR. :smile:

1 Like

@xanni – I’ve reopened and approved. One (very small) suggestion/change. After that, I can merge! :smile:

Merged! :rocket: :blue_heart:

1 Like

Further improvement (probably should have been split into two pull requests in the first place, sorry!)

Move generic instructions into the correct section

@xanni :wave:t4:

Thanks for the second improvement. :smile:

Because the change touches instructions.md rather than just instructions.append.md, it will have to be proposed as a change to problem-specificiations.

I’m fine with changing the instructions.append.md, but we need agreement of at least 3 maintainers before we can make changes to the cross-track instructions.md.

Please open a new thread for it in the general Exercism category (you can tag it improvement or problem-spec, depending).

And you will have to do the instructions.md part of the change in the problem specifications repo, once the discussion has concluded.

Thanks for your understanding!

OK, I have created a new topic here: Moving generic instructions from Python addendum to overall problem specification

I couldn’t see how to tag it though.