Skip to content

Commit 29cc61c

Browse files
lukebaumanncopybara-github
authored andcommitted
Clean up ProfileOptions getattr usages in pathwaysutils.
PiperOrigin-RevId: 926483861
1 parent 3d7f320 commit 29cc61c

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

pathwaysutils/profiling.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -113,8 +113,8 @@ def _is_default_profile_options(
113113
and profiler_options.python_tracer_level
114114
== default_options.python_tracer_level
115115
and profiler_options.duration_ms == default_options.duration_ms
116-
and not getattr(profiler_options, "advanced_configuration", None)
117-
and not getattr(profiler_options, "session_id", None)
116+
and not profiler_options.advanced_configuration
117+
and not profiler_options.session_id
118118
)
119119

120120

@@ -133,9 +133,9 @@ def _create_profile_request(
133133
return profile_request
134134

135135
advanced_config = None
136-
if getattr(profiler_options, "advanced_configuration", None):
136+
if profiler_options.advanced_configuration:
137137
advanced_config = {}
138-
for k, v in getattr(profiler_options, "advanced_configuration").items():
138+
for k, v in profiler_options.advanced_configuration.items():
139139
# Convert python dict to tensorflow.ProfileOptions.AdvancedConfigValue
140140
# json-compatible dict
141141
if isinstance(v, bool):
@@ -169,7 +169,7 @@ def _create_profile_request(
169169
if pw_trace_opts:
170170
xprof_options["pwTraceOptions"] = pw_trace_opts
171171

172-
if getattr(profiler_options, "session_id", None):
172+
if profiler_options.session_id:
173173
xprof_options["traceSessionName"] = profiler_options.session_id
174174

175175
profile_request["xprofTraceOptions"] = xprof_options

0 commit comments

Comments
 (0)