Skip to content

Commit 5e17ccf

Browse files
EddyLXJmeta-codesync[bot]
authored andcommitted
Fix is_using_virtual_table bug in kernel
Differential Revision: D86233647
1 parent 859966c commit 5e17ccf

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

torchrec/distributed/batched_embedding_kernel.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -194,7 +194,7 @@ def _populate_ssd_tbe_params(config: GroupedEmbeddingConfig) -> Dict[str, Any]:
194194
)
195195

196196
# populate init min and max
197-
if config.is_using_virtual_table:
197+
if config.is_using_virtual_table():
198198
_generate_init_range_for_virtual_tables(ssd_tbe_params, config)
199199

200200
if (
@@ -242,7 +242,7 @@ def _populate_ssd_tbe_params(config: GroupedEmbeddingConfig) -> Dict[str, Any]:
242242
)
243243
ssd_tbe_params["cache_sets"] = int(max_cache_sets)
244244

245-
if "kvzch_eviction_tbe_config" in fused_params and config.is_using_virtual_table:
245+
if "kvzch_eviction_tbe_config" in fused_params and config.is_using_virtual_table():
246246
ssd_tbe_params["kvzch_eviction_tbe_config"] = fused_params.get(
247247
"kvzch_eviction_tbe_config"
248248
)
@@ -1975,7 +1975,7 @@ def __init__(
19751975
len({table.embedding_dim for table in config.embedding_tables}) == 1
19761976
), "Currently we expect all tables in SSD TBE to have the same embedding dimension."
19771977
assert (
1978-
config.is_using_virtual_table
1978+
config.is_using_virtual_table()
19791979
), "Try to create ZeroCollisionKeyValueEmbedding for non virtual tables"
19801980
assert embedding_cache_mode == config.enable_embedding_update, (
19811981
f"Embedding_cache kernel is {embedding_cache_mode} "
@@ -2889,7 +2889,7 @@ def __init__(
28892889
len({table.embedding_dim for table in config.embedding_tables}) == 1
28902890
), "Currently we expect all tables in SSD TBE to have the same embedding dimension."
28912891
assert (
2892-
config.is_using_virtual_table
2892+
config.is_using_virtual_table()
28932893
), "Try to create ZeroCollisionKeyValueEmbeddingBag for non virtual tables"
28942894

28952895
for table in config.embedding_tables:

0 commit comments

Comments
 (0)