Skip to content

Commit

Permalink
Frontend changes for adding per_sample_weights in boundary checker (#…
Browse files Browse the repository at this point in the history
…1069)

Summary:
Pull Request resolved: #1069

Follow up for D35768276 (7be1fcb): Spin off the frontend changes in a separate Diff to land later to avoid the BC issue.

Reviewed By: RoshanPAN

Differential Revision: D35882262

fbshipit-source-id: 1d58a8aa53a3f9aa1c36d0926e4d9e4274108783
  • Loading branch information
jianyuh authored and facebook-github-bot committed May 5, 2022
1 parent 9bf4cc4 commit e34d84a
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions fbgemm_gpu/fbgemm_gpu/split_table_batched_embeddings_ops.py
Original file line number Diff line number Diff line change
Expand Up @@ -599,6 +599,7 @@ def forward(
offsets,
self.bounds_check_mode_int,
self.bounds_check_warning,
per_sample_weights,
)
self.step += 1
if len(self.timesteps_prefetched) == 0:
Expand Down Expand Up @@ -1972,6 +1973,7 @@ def forward(
offsets,
self.bounds_check_mode_int,
self.bounds_check_warning,
per_sample_weights,
)
# Note: CPU and CUDA ops use the same interface to facilitate JIT IR
# generation for CUDA/CPU. For CPU op, we don't need weights_uvm and
Expand Down

0 comments on commit e34d84a

Please sign in to comment.