Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Benchmark tridiagonal solver over multiple system sizes #105

Open
wants to merge 62 commits into
base: main
Choose a base branch
from
Open

Benchmark tridiagonal solver over multiple system sizes #105

wants to merge 62 commits into from

Conversation

AdityaDendukuri
Copy link
Collaborator

  1. Benchmark for multiple system sizes.
  2. Added PBS script for benchmarking in etc/derecho/benchmark/

Aditya Dendukuri and others added 30 commits May 22, 2024 13:51
added BLAS library specification in build script
0700222
merged local branch with main
6fee119
Merge branch 'main' of github.com:NCAR/tuv-x
0db8fa3
lapacke added in cmake
42d8147
added google benchmark
c2d6472
run each tridiagonal solver test multiple times and take average of t…
c74f3f2 
…he metrics being recorded
modified plots generated in numerical_analysis
bdcb3a0
include the correct LAPACK package based on the compiler used
b1b5ff4
@AdityaDendukuri @K20shores
Apply suggestions from code review
dee5d7b 
formatting changes

Co-authored-by: Kyle Shores <[email protected]>
seperated tests and benchmark (used google benchmark)
830184d
benchmarking code changes
5e63955
only use diagonally dominant matrices for unit test
3cbb75a
@sjsprecious
Merge branch 'main' into lapacke_debug
2077a2d
@sjsprecious
update docker files to use lapacke for tridiagonal tests
3acc6e4
@sjsprecious
Update ubuntu.yml with lapacke
e035ad9
@sjsprecious
maybe this will work?
cc3403b
@sjsprecious
revert changes to ubuntu.yml
6433ed0
@sjsprecious
Update Dockerfile for CI test
004a55d
@sjsprecious
add missing new line
7e7f02b
@K20shores
allowing benchmark folder to be copied by dockerg
ec88760
@AdityaDendukuri @K20shores
Update cmake/test_util.cmake
63f1e8c 
minor formatting change

Co-authored-by: Kyle Shores <[email protected]>
@K20shores
adding lapacke to ubuntu workflow
7c7c4ed
@K20shores
Merge branch 'lapacke_debug' of https://github.com/NCAR/tuv-x into la…
21b65ca 
…packe_debug
@K20shores
spelling
feb883f
@K20shores
serial build
ab61424
@K20shores
undoing cmake formatting changes
6fc7973
@K20shores
using find lapacke from other open source libraries
783b494
@K20shores
correcting working directory
4dc105f
@K20shores
adding benchmark cmake variable
6621164
@K20shores
dash
26fd879
Aditya Dendukuri and others added 24 commits July 3, 2024 10:41
fixed bug in test_error_function unit test; updated documentation
510c956
Merge branch 'lapacke_debug' of github.com:NCAR/tuv-x into lapacke_debug
9820583
uncommented findLAPACKE
9f54f5a
added some more documentation
9f983b3
fixed a bug that caused the memcheck to fail
c22d41d
@AdityaDendukuri @mwaxmonsky
Apply suggestions from code review
eda38a6 
CMAKE file formatting changes

Co-authored-by: mwaxmonsky <[email protected]>
switched to std::abs instead of ::abs
d3b66f7
Merge branch 'lapacke_debug' of github.com:NCAR/tuv-x into lapacke_debug
9f256bd
reverted formatting changes in cmake file
bb1c227
re-added lapacke dependency in test_util
05e2827
linked LAPACKE libraries to unit tests
04a9b0f
removed typedefs in test_error_function.cpp
20e73de
@AdityaDendukuri
updated code documentation
34b529b
@AdityaDendukuri
documentaion changes
88fa5a6
@AdityaDendukuri
documentation changes
302fc37
documentation changes
73ac682
switched back to version 0.9.0
1d165a8
Documentation Changes
23b358e
@AdityaDendukuri @K20shores
Update src/CMakeLists.txt
a572368 
Co-authored-by: Kyle Shores <[email protected]>
@AdityaDendukuri @K20shores
Update src/CMakeLists.txt
38e9903 
Co-authored-by: Kyle Shores <[email protected]>
rolled back format changes
b6a10f8
Merge branch 'lapacke_debug' of github.com:NCAR/tuv-x into lapacke_debug
766ecf8
Merge branch 'main' of github.com:NCAR/tuv-x
e3fdca2
benchmark over multiple sizes
1fc121f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjsprecious sjsprecious Awaiting requested review from sjsprecious

@mattldawson mattldawson Awaiting requested review from mattldawson

@K20shores K20shores Awaiting requested review from K20shores

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AdityaDendukuri @sjsprecious @K20shores