Skip to content

Commit

Permalink
[OpPerf] Fixed Python profiler bug (apache#17642)
Browse files Browse the repository at this point in the history
* Changed arg structure in op func call

* Length check to prevent index out of bounds error

* Dropping args[2] as it is no longer used (only using kwargs)
  • Loading branch information
connorgoggins authored Feb 26, 2020
1 parent c141867 commit 1906eff
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion benchmark/opperf/utils/profiler_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -248,7 +248,7 @@ def python_profile(func):
@functools.wraps(func)
def python_profile_it(*args, **kwargs):
runs = args[1]
modified_args = (args[0], 1, args[2])
modified_args = (args[0], 1)
times = []

for _ in range(runs):
Expand Down

0 comments on commit 1906eff

Please sign in to comment.