-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[Doc] Separate arithmetics.rst from syntax.rst and fill it with more details #1761
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1761 +/- ##
==========================================
- Coverage 43.19% 42.75% -0.44%
==========================================
Files 44 44
Lines 6075 6132 +57
Branches 1050 1070 +20
==========================================
- Hits 2624 2622 -2
- Misses 3297 3356 +59
Partials 154 154
Continue to review full report at Codecov.
|
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.
🚀 as always.
Builtin-alike functions | ||
*********************** | ||
|
||
.. function:: abs(x) | ||
.. function:: max(x, y, ...) | ||
.. function:: min(x, y, ...) | ||
.. function:: pow(x, y) | ||
|
||
Same as ``x ** y``. |
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.
I'm kinda confused here. What does "Builtin-alike" mean? 🤔 Would just "built-in" be enough?
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.
Since abs
and max
are builtin functions, but they are replaced by ti.ti_abs
and ti.ti_max
by @ti.kernel
, which is our functions. They just share the same interface with the Python built-in ones.
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.
All good except needs to determine the Python/C style line :) Thanks!
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.
LGTM! I can't find anything that needs to be changed 👍
Related issue = #
[Click here for the format server]
So that
syntax.rst
can focus on explaining Taichi kernels and functions.I don't rename
syntax.rst
in afraid of broken links and spoil translators' work insyntax.po
.