Skip to content

Commit

Permalink
Inform asv that there should be no warmup runs for time_remove benchmark
Browse files Browse the repository at this point in the history
Thanks to @trexfeathers for the RTFM pointers in
airspeed-velocity/asv#1347 (comment)
  • Loading branch information
yarikoptic committed Oct 9, 2023
1 parent a96c51c commit 1169180
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions benchmarks/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,8 @@ def time_uninstall(self):
self.ds.drop(subm["path"], recursive=True, what='all',
reckless='kill')

# disabled since apparently setup is not running for each execution!
# TODO: fix up after hearing on https://github.com/airspeed-velocity/asv/issues/1347
#def time_remove(self):
# self.ds.drop(what='all', reckless='kill', recursive=True)
def time_remove(self):
self.ds.drop(what='all', reckless='kill', recursive=True)

def time_diff(self):
self.ds.diff(fr="HEAD^")
Expand All @@ -120,3 +118,6 @@ def time_status(self):

def time_status_recursive(self):
self.ds.status(recursive=True)


supers.time_remove.warmup_time = 0

0 comments on commit 1169180

Please sign in to comment.