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

Introduce a temporary boolean constant for benchmarking #657

Merged
merged 2 commits into from
Mar 25, 2020

Conversation

xumingkuan
Copy link
Collaborator

Related issue id = #656

I'm now using test_ad_if_mutable in test_ad_if.py for benchmarking. It already reduced the number of statements from 96 to 92 :)

[Click here for the format server]

@xumingkuan
Copy link
Collaborator Author

Why does @taichi-gardener modify files I didn't change even if what I clicked is exactly this PR?

@yuanming-hu
Copy link
Member

I'm now using test_ad_if_mutable in test_ad_if.py for benchmarking. It already reduced the number of statements from 96 to 92 :)

Cool! Thanks.

Why does @taichi-gardener modify files I didn't change even if what I clicked is exactly this PR?

Probably not your fault. I'll take a look at this.

If this PR ready for review? Please simply request a review from me when you are ready. Thanks.

@xumingkuan
Copy link
Collaborator Author

It's ready for review.

I googled how to request a review on GitHub, and GitHub Help showed

In the list of pull requests, click the pull request that you'd like to ask a specific person or a team to review.

Navigate to Reviewers in the right sidebar.

To request a review from a suggested person under Reviewers, next to their username, click Request.

However, my right sidebar only shows

Reviewers
No reviews

without a button to click. Why is that?

@yuanming-hu yuanming-hu self-requested a review March 25, 2020 23:01
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.

Looks good. Thanks!

@yuanming-hu
Copy link
Member

It's ready for review.

I googled how to request a review on GitHub, and GitHub Help showed

In the list of pull requests, click the pull request that you'd like to ask a specific person or a team to review.
Navigate to Reviewers in the right sidebar.
To request a review from a suggested person under Reviewers, next to their username, click Request.

However, my right sidebar only shows

Reviewers
No reviews

without a button to click. Why is that?

interesting... Do you see a gear there?
Screenshot from 2020-03-25 19-04-06

@yuanming-hu yuanming-hu merged commit 068af03 into taichi-dev:master Mar 25, 2020
@xumingkuan
Copy link
Collaborator Author

It's ready for review.
I googled how to request a review on GitHub, and GitHub Help showed

In the list of pull requests, click the pull request that you'd like to ask a specific person or a team to review.
Navigate to Reviewers in the right sidebar.
To request a review from a suggested person under Reviewers, next to their username, click Request.

However, my right sidebar only shows

Reviewers
No reviews

without a button to click. Why is that?

interesting... Do you see a gear there?
Screenshot from 2020-03-25 19-04-06

No, I don't. However, I can re-request a review from you after you self-requested the review and before approving.

@xumingkuan
Copy link
Collaborator Author

Oh, I saw this on the website above:

Pull request authors can't request reviews unless they are either a repository owner or collaborator with write access to the repository.

@yuanming-hu
Copy link
Member

I'll assume non-Draft PRs are ready for review next time. Thanks for the info.

@xumingkuan xumingkuan deleted the benchmark branch March 26, 2020 03:22
@archibate
Copy link
Collaborator

Oh, I saw this on the website above:

Pull request authors can't request reviews unless they are either a repository owner or collaborator with write access to the repository.

I see a little gear on the right of Reviewers... was that expected? I think I don't have a write access?

@yuanming-hu
Copy link
Member

yuanming-hu commented Mar 26, 2020

Oh, I saw this on the website above:

Pull request authors can't request reviews unless they are either a repository owner or collaborator with write access to the repository.

I see a little gear on the right of Reviewers... was that expected? I think I don't have a write access?

Since you have made significant contributions, your role has been promoted from Read to Triage, which means you have the right to manage issues and pull requests :-) That's why you can request reviewers.

Ultimately I think you will get Write access to the repo. But please keep in mind that with great power comes great responsibilities, and please lead by example :-)

Screenshot from 2020-03-26 01-09-24

archibate pushed a commit to archibate/taichi that referenced this pull request Mar 31, 2020
…ced optimizations (taichi-dev#657)

* Introduce a temporary boolean constant for benchmarking

* [skip ci] enforce code format

Co-authored-by: Taichi Gardener <taichigardener@gmail.com>
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.

4 participants