Skip to content

Commit

Permalink
Fixed processed bytes calculation in APPLY_BOOLEAN_MASK benchmark. (#…
Browse files Browse the repository at this point in the history
…13937)

Due to missing parentheses in APPLY_BOOLEAN_MASK benchmark, the number of written bytes were not multiplied by the number of iterations of this benchmark.

This patch relates to #13735.

Authors:
  - Martin Marenz (https://github.com/Blonck)

Approvers:
  - Nghia Truong (https://github.com/ttnghia)
  - David Wendt (https://github.com/davidwendt)

URL: #13937
  • Loading branch information
Blonck authored Aug 23, 2023
1 parent 88a8efc commit e16ed81
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cpp/benchmarks/stream_compaction/apply_boolean_mask.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ void calculate_bandwidth(benchmark::State& state, cudf::size_type num_columns)
(column_bytes_out + validity_bytes_out) * num_columns; // writing columns

state.SetItemsProcessed(state.iterations() * column_size * num_columns);
state.SetBytesProcessed(static_cast<int64_t>(state.iterations()) * bytes_read + bytes_written);
state.SetBytesProcessed(static_cast<int64_t>(state.iterations()) * (bytes_read + bytes_written));
}

} // namespace
Expand Down

0 comments on commit e16ed81

Please sign in to comment.