-
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] Revise doc for GUI system. #4006
Conversation
✔️ Deploy Preview for jovial-fermat-aa59dc ready! 🔨 Explore the source changes: fc67ab9 🔍 Inspect the deploy log: https://app.netlify.com/sites/jovial-fermat-aa59dc/deploys/61dfc6c25ccd5b0007fe16fc 😎 Browse the preview: https://deploy-preview-4006--jovial-fermat-aa59dc.netlify.app |
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.
Minor editorial updates.
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/taichi-api-docs-preview/taichi/A4xEdu4sxKgytuRiyAQN21UNvphA [Deployment for 319a960 canceled] |
✔️ Deploy Preview for docsite-preview ready! 🔨 Explore the source changes: 319a960 🔍 Inspect the deploy log: https://app.netlify.com/sites/docsite-preview/deploys/61efc51715fbb40008a6b69e 😎 Browse the preview: https://deploy-preview-4006--docsite-preview.netlify.app |
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.
Just few nits.
can detect the keys you pressed. It must be used together with `gui.get_event`, or it won't be updated! For | ||
example: | ||
`gui.is_pressed(key, ...)` detects the keys you pressed. You must use it | ||
together with `gui.get_event`. Otherwise, it is not updated. For example: |
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.
Will it won't be updated
be more clear?
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.
This is simply a style issue. Avoid future tense and contraction in technical writing.
@@ -442,8 +247,8 @@ while gui.running: | |||
|
|||
## Image I/O | |||
|
|||
[`ti.imwrite(img, filename)`](https://api-docs.taichi.graphics/src/taichi.misc.html?highlight=imwrite#taichi.misc.image.imwrite) | |||
can export a `np.ndarray` or Taichi field (`ti.Matrix.field`, `ti.Vector.field`, or `ti.field`) to a specified location `filename`. | |||
`ti.imwrite(img, filename)` exports an `np.ndarray` or a Taichi field |
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.
Maybe we don't need an an
but an a
for np.ndarray
?
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.
It ought to be "an" here. The pronunciation of np is /'en'pi:: /, which starts with a vowel.
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!
Hi, thanks for this update. Could you add the following topics to the content?
|
I have addressed numpy and fps part in the latest commit. |
/format |
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!
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!
The subsection Also, shall we expand the subsection |
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!
Shrunk GUI doc a lot. This is my first revision. What's everybody else's opinions?