Skip to content

Commit

Permalink
Fix the data dist benchmark binary name to use "var" naming convention
Browse files Browse the repository at this point in the history
Change the deployed binary to "write_var_data_dist" so it matches
the configured test reference in h5bench.
The binary name needs to match the test name so that it can be called
by the hbench wrapper.
This follows convention of "write_var_normal_dist".
Update code to log the correct benchmark name.
  • Loading branch information
jprorama committed Aug 20, 2024
1 parent 9cf8232 commit de1d787
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -134,8 +134,8 @@ target_link_libraries(h5bench_write_var_normal_dist h5bench_util hdf5 z m ${CMAK

set(h5bench_write_data_dist_src h5bench_patterns/h5bench_write_data_dist.c)

add_executable(h5bench_write_data_dist ${h5bench_write_data_dist_src})
target_link_libraries(h5bench_write_data_dist h5bench_util hdf5 z m ${CMAKE_DL_LIBS} MPI::MPI_C)
add_executable(h5bench_write_var_data_dist ${h5bench_write_data_dist_src})
target_link_libraries(h5bench_write_var_data_dist h5bench_util hdf5 z m ${CMAKE_DL_LIBS} MPI::MPI_C)

# h5bench WRITE ###############################################################
#
Expand Down Expand Up @@ -365,7 +365,7 @@ install(
h5bench_write
h5bench_write_unlimited
h5bench_write_var_normal_dist
h5bench_write_data_dist
h5bench_write_var_data_dist
h5bench_overwrite
h5bench_append
h5bench_read
Expand Down
2 changes: 1 addition & 1 deletion h5bench_patterns/h5bench_write_data_dist.c
Original file line number Diff line number Diff line change
Expand Up @@ -1277,7 +1277,7 @@ main(int argc, char *argv[])

if (params.useCSV) {
fprintf(params.csv_fs, "metric, value, unit\n");
fprintf(params.csv_fs, "operation, %s, %s\n", "write_var_normal_dist", "");
fprintf(params.csv_fs, "operation, %s, %s\n", "write_var_data_dist", "");
fprintf(params.csv_fs, "ranks, %d, %s\n", NUM_RANKS, "");
fprintf(params.csv_fs, "Total number of particles, %lldM, %s\n", TOTAL_PARTICLES / (M_VAL), "");
fprintf(params.csv_fs, "Final mean particles, %ld, %s\n", final_mean, "");
Expand Down

0 comments on commit de1d787

Please sign in to comment.