Skip to content

Commit

Permalink
Fix incorrect proc count on ruby in performance test, fix bug in perf…
Browse files Browse the repository at this point in the history
…ormance_analysis.py
  • Loading branch information
ldowen committed Jan 16, 2025
1 parent daeb774 commit 7225834
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions scripts/devtools/performance_analysis.py
Original file line number Diff line number Diff line change
Expand Up @@ -271,11 +271,11 @@ def get_caliper_files(file_path):
if (main_diff > ref_thresh):
cur_status = "FAILED"
if args.display:
display(cprof.statsframe.tree(metric1, metric1+"_mean"))
display(ctest.statsframe.tree(metric1, metric1+"_mean"))
elif (main_diff < -ref_thresh):
cur_status = "PASSED"
if args.display:
display(cprof.statsframe.tree(metric1, metric1+"_mean"))
display(ctest.statsframe.tree(metric1, metric1+"_mean"))
else:
cur_status = "PASSED"
test_status.update({test_name: (cur_status, cur_main, ref_main, ref_thresh)})
Expand Down
2 changes: 1 addition & 1 deletion tests/performance.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
# Retrieve the host name and remove any numbers
temp_uname = os.uname()
hostname = "".join([i for i in temp_uname[1] if not i.isdigit()])
mac_procs = {"rzhound": 112, "rzwhippet": 112, "ruby": 112,
mac_procs = {"rzhound": 112, "rzwhippet": 112, "ruby": 56,
"rzadams": 84, "rzvernal": 64, "tioga": 64,
"rzansel": 40, "lassen": 40, "rzgenie": 36}
# Find out how many nodes our allocation has grabbed
Expand Down

0 comments on commit 7225834

Please sign in to comment.