Skip to content

Commit

Permalink
Revert D66508492: Set remote cache version and backend type once in c…
Browse files Browse the repository at this point in the history
…ompilation metrics

Differential Revision:
D66508492

Original commit changeset: 014722ba2cfb

Original Phabricator Diff: D66508492

fbshipit-source-id: 9a27483aa5c6fd54f21fbbb9d56844e600cee3dc
  • Loading branch information
malfet authored and facebook-github-bot committed Dec 3, 2024
1 parent c07f33b commit 711de8c
Showing 1 changed file with 0 additions and 20 deletions.
20 changes: 0 additions & 20 deletions userbenchmark/dynamo/dynamobench/_dynamo/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -996,24 +996,6 @@ def safe_str(item: Any) -> str:
return ",".join(safe_str(item) for item in metric)

structured_logging_overhead_s = torch._logging.get_structured_logging_overhead()

if torch._inductor.utils.should_use_remote_fx_graph_cache():
try:
from torch._inductor.fb.remote_cache import (
FbRemoteFxGraphCache,
REMOTE_CACHE_VERSION,
)
except ModuleNotFoundError:
REMOTE_CACHE_VERSION = None
inductor_fx_remote_cache_backend_type = None

remote_cache_version = REMOTE_CACHE_VERSION
backend = FbRemoteFxGraphCache.get_remote_backend()
inductor_fx_remote_cache_backend_type = type(backend).__name__
else:
inductor_fx_remote_cache_backend_type = None
remote_cache_version = None

common_metrics = {
"compile_id": str(torch._guards.CompileContext.current_compile_id()),
"start_time_us": start_time_ns // 1000,
Expand All @@ -1031,8 +1013,6 @@ def safe_str(item: Any) -> str:
"inductor_fx_remote_cache_miss_keys": _convert_collection_to_str(
"inductor_fx_remote_cache_miss_keys"
),
"remote_cache_version": remote_cache_version,
"inductor_fx_remote_cache_backend_type": inductor_fx_remote_cache_backend_type,
}

# TODO: The following are legacy fields, populated from the fields that replace
Expand Down

0 comments on commit 711de8c

Please sign in to comment.