Skip to content

Commit

Permalink
Comparator fix
Browse files Browse the repository at this point in the history
  • Loading branch information
geoffxy committed Nov 1, 2023
1 parent d519db5 commit 1b048f4
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/brad/planner/compare/cost.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,8 @@ def best_cost_under_perf_ceilings(
) -> BlueprintComparator:
def is_better_than(left: ComparableBlueprint, right: ComparableBlueprint) -> bool:
# Check transactional latency ceilings first.
left_txn_p90 = left.get_predicted_transactional_latencies()[0]
right_txn_p90 = right.get_predicted_transactional_latencies()[0]
left_txn_p90 = left.get_predicted_transactional_latencies()[1]
right_txn_p90 = right.get_predicted_transactional_latencies()[1]

# If one of these candidates have NaN predictions, we need to
# consider other factors. NaN indicates that a prediction is not
Expand Down

0 comments on commit 1b048f4

Please sign in to comment.