-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
[stdlib] Cleanup polynomial.mojo
, improve docstring
#3066
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you! Do you mind fixing up the merge conflicts and then I'm happy to merge this?
b8f83ba
to
15322f7
Compare
polynomial.mojo
off unroll
polynomial.mojo
, improve docstring
!sync |
15322f7
to
d641d87
Compare
- Small simplification to the function `_horner_evaluate`. - Improve its docstring to match the argument names. - Finish off removing `unroll` by cleaning up the imports. Signed-off-by: Yiwu Chen <210at85@gmail.com>
d641d87
to
445fb76
Compare
This should be a trivial cleanup, however, inlining This is the output of the following code: print("alias", c_second_from_last, coefficients[num_coefficients - 2])
|
Signed-off-by: Yiwu Chen <210at85@gmail.com>
Can you file a compiler bug for this, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In order to not have this be blocked by the compiler bug, I'm going to to merge the docstring cleanups here and leave the body as-is.
!sync |
✅🟣 This contribution has been merged 🟣✅ Your pull request has been merged to the internal upstream Mojo sources. It will be reflected here in the Mojo repository on the nightly branch during the next Mojo nightly release, typically within the next 24-48 hours. We use Copybara to merge external contributions, click here to learn more. |
Landed in 7ed5407! Thank you for your contribution 🎉 |
[External] [stdlib] Improve docstring in `polynomial.mojo` Improve the `_horner_evaluate` docstring to match the argument names. Co-authored-by: soraros <soraros@users.noreply.github.com> Closes #3066 MODULAR_ORIG_COMMIT_REV_ID: 228e5b772e138765b9e2d73a8f5f1bead515661d
_horner_evaluate
.Finish off removingunroll
by cleaning up the imports.