Skip to content
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

[Lang] Improve ti example so that users can choose which example to run by entering numbers. #5265

Merged
merged 9 commits into from
Jul 7, 2022

Conversation

neozhaoliang
Copy link
Contributor

Improve the ti example command so that users can choose which example to run by entering numbers.

@netlify
Copy link

netlify bot commented Jun 27, 2022

Deploy Preview for docsite-preview ready!

Name Link
🔨 Latest commit 79b1c93
🔍 Latest deploy log https://app.netlify.com/sites/docsite-preview/deploys/62c5b5cf07961500099b0e42
😎 Deploy Preview https://deploy-preview-5265--docsite-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@neozhaoliang neozhaoliang changed the title Fix examples [Lang] Improve ti example so that users can choose which example to run by entering numbers. Jun 27, 2022
@neozhaoliang neozhaoliang requested review from victoriacity, 0xzhang and yuanming-hu and removed request for victoriacity and 0xzhang July 6, 2022 09:17
Copy link
Member

@yuanming-hu yuanming-hu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with some nits! Thank you!

python/taichi/_main.py Show resolved Hide resolved
python/taichi/_main.py Outdated Show resolved Hide resolved
neozhaoliang and others added 2 commits July 7, 2022 00:17
Co-authored-by: Yuanming Hu <yuanming-hu@users.noreply.github.com>
@neozhaoliang neozhaoliang merged commit 67cee8b into taichi-dev:master Jul 7, 2022
@neozhaoliang neozhaoliang deleted the fix-examples branch July 7, 2022 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants