-
Notifications
You must be signed in to change notification settings - Fork 43
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
update num_tasks
in click_through_rate metric
#64
Conversation
This pull request was exported from Phabricator. Differential Revision: D40402724 |
Codecov Report
@@ Coverage Diff @@
## main #64 +/- ##
==========================================
+ Coverage 94.37% 94.39% +0.01%
==========================================
Files 117 117
Lines 6309 6328 +19
==========================================
+ Hits 5954 5973 +19
Misses 355 355
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Summary: Pull Request resolved: pytorch#64 click_through_rate metric was introduced by diff , however, I missed that the num_tasks was not included in the original implementation during my review. This diff updates this information. Reviewed By: bobakfb Differential Revision: D40402724 fbshipit-source-id: 9bfb0635a348e33d4c1064b3b38412bc1a846438
f53698d
to
64b4c4d
Compare
This pull request was exported from Phabricator. Differential Revision: D40402724 |
Summary: Pull Request resolved: pytorch#64 click_through_rate metric was introduced by diff , however, I missed that the num_tasks was not included in the original implementation during my review. This diff updates this information. Reviewed By: bobakfb Differential Revision: D40402724 fbshipit-source-id: 4bfb7951df12a6b7e5fb7823ba26ac4cbfc70bd3
64b4c4d
to
92812d0
Compare
This pull request was exported from Phabricator. Differential Revision: D40402724 |
Summary: Pull Request resolved: pytorch#64 click_through_rate metric was introduced by diff , however, I missed that the num_tasks was not included in the original implementation during my review. This diff updates this information. Reviewed By: bobakfb Differential Revision: D40402724 fbshipit-source-id: 0ac7b73d87e1189876ca55989cb1e9d5d5f0db26
92812d0
to
f503b84
Compare
This pull request was exported from Phabricator. Differential Revision: D40402724 |
Summary:
click_through_rate metric was introduced by diff , however, I missed that the num_tasks was not included in the original implementation during my review.
This diff updates this information.
Reviewed By: bobakfb
Differential Revision: D40402724