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

Torchrec gpu stable #660

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

czxttkl
Copy link
Contributor

@czxttkl czxttkl commented Jul 27, 2022

Summary: use this

Differential Revision: D38185498

@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D38185498

@codecov-commenter
Copy link

codecov-commenter commented Jul 27, 2022

Codecov Report

Merging #660 (d1b8a4a) into main (1002d12) will increase coverage by 17.90%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #660       +/-   ##
===========================================
+ Coverage   69.27%   87.17%   +17.90%     
===========================================
  Files         354      355        +1     
  Lines       22586    22642       +56     
  Branches       44       44               
===========================================
+ Hits        15647    19739     +4092     
+ Misses       6913     2877     -4036     
  Partials       26       26               
Impacted Files Coverage Δ
reagent/mab/ucb.py 86.84% <0.00%> (-2.64%) ⬇️
reagent/test/test_data/ex_mdps.py 96.42% <0.00%> (ø)
reagent/core/types.py 86.75% <0.00%> (+0.19%) ⬆️
reagent/prediction/predictor_wrapper.py 75.54% <0.00%> (+0.62%) ⬆️
reagent/training/dqn_trainer_base.py 95.52% <0.00%> (+0.74%) ⬆️
reagent/models/synthetic_reward.py 97.20% <0.00%> (+0.93%) ⬆️
reagent/core/tracker.py 93.97% <0.00%> (+3.61%) ⬆️
reagent/core/aggregators.py 94.54% <0.00%> (+3.63%) ⬆️
reagent/preprocessing/postprocessor.py 96.15% <0.00%> (+3.84%) ⬆️
reagent/core/utils.py 85.10% <0.00%> (+4.25%) ⬆️
... and 80 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1002d12...d1b8a4a. Read the comment docs.

@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D38185498

czxttkl added a commit to czxttkl/ReAgent that referenced this pull request Jul 27, 2022
Summary:
Pull Request resolved: facebookresearch#660

use this

Differential Revision: D38185498

fbshipit-source-id: 72785c9d62ddcc14d2a8529268261137e74b30a2
czxttkl added a commit to czxttkl/ReAgent that referenced this pull request Jul 27, 2022
Summary:
Pull Request resolved: facebookresearch#660

use this

Differential Revision: D38185498

fbshipit-source-id: 409aa190e0a0af00d6e99fce2852b8087623216b
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D38185498

Summary:
Pull Request resolved: facebookresearch#660

use this

Differential Revision: D38185498

fbshipit-source-id: 16dd4698709687a441253630f2bb5db885568c81
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D38185498

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants