-
Notifications
You must be signed in to change notification settings - Fork 52
Use caliper #137
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.
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
Open
jonesholger
wants to merge
205
commits into
llnl:develop
Choose a base branch
from
jonesholger:use_caliper
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Use caliper #137
Changes from 45 commits
Commits
Show all changes
205 commits
Select commit
Hold shift + click to select a range
c64db24
Initial link against spack built caliper; ATOMIC_PI:BASE_OpenMP instr…
jonesholger 2db7cb1
suppress dirty blt
jonesholger d7ba2ad
Caliper timer start/stop moved into kernel base so it covers all of t…
jonesholger 940a07e
exclude warmup; add adiak and other configuration meta-data support
jonesholger 513455d
.gitignore update for vim swap files
jonesholger e30a1b7
change rajaperf_config so it's compatible with Caliper; Adiak::value …
jonesholger c6e4771
per kernel.variant json files with additional adiak keys kernel, vari…
jonesholger 90be795
change output to spot cali files, and generic kernel name for region
jonesholger fc1b920
Initial Caliper manager per variant
jonesholger d681a64
Caliper tree is now variant->group->kernel
jonesholger 516507d
compiler_suffic adiak key; cleanup
jonesholger 7914e24
Add perf.py Hatchet Notebook with GenericFrame class; allows comparis…
jonesholger 8e08cd5
add some notes on how to pickup python caliperreader; remove default_…
jonesholger 01d79b7
Merge branch 'develop' into use_caliper
jonesholger dc55482
add compiler version via cmake and compiler path version via regex of…
jonesholger af35eee
changed example perf notebook to use new compiler key and rename note…
jonesholger 130dfb2
Add ExtractCommonSubtree to perf.py to support comparing trees which …
jonesholger b5ad2e4
use hatchet caliper native reader
jonesholger b1275b7
Merge branch 'develop' into use_caliper
jonesholger e89f97d
fix warmup logic; add adiak ProblemSize and SizeMeaning keys
jonesholger e7577d1
perf.py supports running sweeps
jonesholger b19a19f
Merge branch 'develop' into use_caliper
jonesholger 76aff45
Merge branch 'bugfix/burmark1/HEAT_3D_index_mapping' into use_caliper
jonesholger 64d5219
import of adiak works for adiak and adiak::adiak library exports; cle…
jonesholger 0e9a90e
Merge branch 'develop' into use_caliper
jonesholger 61dae2a
remove hatchet based scripts until next hatchet/spot release; this wi…
jonesholger 398ba56
Merge branch 'develop' into use_caliper
jonesholger c617652
Make sweep_size script more flexible
MrBurmark 2933af1
Add graphing script to go with sweep script.
MrBurmark 1b63fe2
Add more help in toss4 cray mvapich amdclang script
MrBurmark 0ce8c63
Merge remote-tracking branch 'origin/develop' into feature/burmark1/g…
MrBurmark eb54c42
Add names to sweep_size script
MrBurmark d8afe7d
Remove extra text in graph names
MrBurmark 29a3873
Change color factors to avoid saturating base
MrBurmark 93faac1
Fix typos in sweep_size
MrBurmark 55ed8be
Add simple throughput to graph script
MrBurmark 699ff16
Add more formatting to graphs
MrBurmark 2b8b5ce
Add args to include or exclude kernels from graphing
MrBurmark c256961
Avoid recounting run sizes
MrBurmark 85b3d2c
fixup graph naming
MrBurmark 41983b1
Add options to include/exclude variants and tunings
MrBurmark 074c208
Fix issue with repeated run sizes
MrBurmark 63e1ffd
Add print of sweeps
MrBurmark ced724c
Refactor sweep_graph to use some classes
MrBurmark 71a9451
Add templated data computation
MrBurmark dd1bd5c
plot on same graph, add linear regression
MrBurmark 5dd7d4b
Add ability to graph bar charts across kernels
MrBurmark 59cc13c
Clarify segmented linear regression breakpoints
MrBurmark a1e172e
Add option to exclude sweeps
MrBurmark c488212
Add kernel groups to simplify plotting
MrBurmark 2098f85
Add more generic data tree class
MrBurmark 1e3824c
remove use test tuning
MrBurmark 0d3e5be
rename some variables
MrBurmark e2eb0bf
Switched to more flexible copmute scheme
MrBurmark 059e4df
Generalize graphing implementation
MrBurmark 4a176b6
Fix division by 0 error
MrBurmark a26f65f
fix split line plotting
MrBurmark 0f2709f
Add more color utilities
MrBurmark 8cde548
fixup arg handling
MrBurmark 5beb2b7
Add ability to change graph labels
MrBurmark 9f7a00f
Add and update kind templates
MrBurmark ed0621f
Add histogram plotting
MrBurmark 187e7d1
Add option to move legend
MrBurmark 700be35
Move bar zorder to under axes
MrBurmark 19ac08a
Merge branch 'develop' into pr-from-fork/137
rhornung67 16b8340
Code formatting
rhornung67 9295555
Fix merge errors
rhornung67 a40b276
Merge branch 'develop' into pr-from-fork/137
rhornung67 bc0573a
Update CMakeLists.txt
rhornung67 2a761bb
Update src/basic/PI_ATOMIC-OMP.cpp
rhornung67 e79cf13
Update src/basic/PI_ATOMIC-OMP.cpp
rhornung67 81e29e4
Update .gitmodules
rhornung67 53671b0
Update src/basic/PI_ATOMIC-OMP.cpp
rhornung67 a4c12a8
Merge branch 'develop' into use_caliper
jonesholger 88d98ed
Merge branch 'use_caliper' of https://github.com/jonesholger/RAJAPerf…
jonesholger 14f0a39
add +caliper spec to package.py; add include_directories for adiak
jonesholger 3b2de08
change set_target_properties for adiak target to for +cuda spec; we …
jonesholger 39fde9b
Merge branch 'use_caliper' into pr-from-fork/137
jonesholger 52ac292
gen per kernel adiak keyval such as runreps, itersperrep, etc
jonesholger a4aec2b
add adiak_ prefix to rajaperf::configuration variables in caliper sec…
jonesholger 84aedca
remove compiler info noise
jonesholger 248ffd3
update spack package to use caliper@2.8.0; also add +rocm to caliper …
jonesholger 8f32df7
fixup cmake define to pickup caliper/adiak includes in src/CMakeLists…
jonesholger 763c358
Merge branch 'pr-from-fork/137' into feature/holger/graph_sweeps_caliper
jonesholger 7ec27ce
support output directory -od flag for spot output files
jonesholger b73caaa
Merge branch 'pr-from-fork/137' into feature/holger/graph_sweeps_caliper
jonesholger 08ea0f9
add support to process caliper timing info; still uses orig runinfo m…
jonesholger 052ca63
per kernel column metadata; currently exported as strings since some …
jonesholger b8041e7
Merge branch 'feature/burmark1/graph_sweeps' into caliper_metadata_at…
jonesholger 1950e27
fixup xdata sort order using numpy
jonesholger ef58b0c
switch kernel attribute metadata datatype to double
jonesholger fffe3e0
Merge branch 'caliper_metadata_attribute' into feature/holger/graph_s…
jonesholger 4ad948f
modify adiak runinfo metadata so it can be used in python scripts as …
jonesholger ba525df
add checks for include/exclude kernels and error out in case of cmdli…
jonesholger 775701c
fix bug in read_caliper_runinfo_size removing errant continue in run_…
jonesholger b59755e
add legend order routine - currently developer centric as order is fi…
jonesholger f5f6a35
fix doOnceCaliMeta for multiple tunings
jonesholger f444177
parent/child parsers; add --directories; add prescan dict to parser n…
jonesholger 38bc905
continue integration of argparse_sweep_graph module
jonesholger ecc8860
Merge branch 'develop' into pr-from-fork/137
jonesholger 0a9bef0
adiak runinfo metadata is wrapped in dict so we can use eval in pytho…
jonesholger e512b0d
split off data classes into separate module data_classes_sweep_graph;…
jonesholger 57f50c4
change column attribute data such as ProblemSize so aggregation is mo…
jonesholger bf45a79
cmake Caliper/Adiak includes back in main so it's in one section, and…
jonesholger a616fe1
put cmake Caliper/Adiak includes back into top; accomodate Adiak@0.2.…
jonesholger 8c118ff
Merge branch 'use_argparse' of github.com:jonesholger/RAJAPerf into u…
jonesholger 5941881
Merge branch 'use_argparse' into pr-from-fork/137
jonesholger 505cc95
Empty commit to force CI
rhornung67 3002f99
Remove README banner that we don't want in develop
rhornung67 10c5480
add caliper based prescan
jonesholger b19939e
Merge branch 'develop' into use_caliper
rhornung67 f91cd21
fix some namespace bugs wrt Data class; add kinds closeness check in …
jonesholger f57e3c7
move several free functions back into Data class - reorg not apropo y…
jonesholger 1cc3f4a
still need our local cr (caliper reader) object to pass to prescan
jonesholger d590132
Merge branch 'use_argparse' into pr-from-fork/137
jonesholger 4800c32
Merge branch 'pr-from-fork/137' of github.com:LLNL/RAJAPerf into pr-f…
jonesholger 334bd37
add --recolor --reformat flags; kind closeness check is more insensit…
jonesholger 2e03612
Caliper column metadata Aggregatable attributes need to be initialize…
jonesholger a353d4f
Merge branch 'use_argparse' into pr-from-fork/137
jonesholger 0171719
add --recolor --reformat flags; kind closeness check is more insensit…
jonesholger 7c6f110
Caliper column metadata Aggregatable attributes need to be initialize…
jonesholger 9ad36ac
Spot config add via command line argument implemented, e.g. you can s…
jonesholger 62341ea
add help text for --add-to-spot-config -atsc
jonesholger 1d2e427
Merge branch 'caliper-value-add' into pr-from-fork/137
jonesholger 93dee34
advance caliper to caliper@master until caliper@2.9.0 is released
jonesholger 8985613
Merge branch 'pr-from-fork/137' into use_caliper
jonesholger dfbdcb9
use cmp0048 for project version, and propagate to original version va…
jonesholger a742ebe
remove dirty git ignore for BLT in .gitmodules; add README.md documen…
jonesholger ef804d4
add Use Caliper to table of contents in README.md
jonesholger 752f643
adjust for github render
jonesholger ccdfc0e
Add event-trace info for CUDA and ROCM; to be viewed in chrome or per…
jonesholger 1ec65b5
noticed an issue with rendering text between < > brackets on github
jonesholger d5dbda0
minor edits to README -- mostly format/highlight tags
jonesholger 72eb5e9
Merge branch 'develop' into use_caliper
jonesholger 3ae59ac
advance spack commit in uberenv config to v0.19.0 via hash
jonesholger 22c8f5b
setup new adiak key ProblemSizeRunParam to reflect command line arg -…
jonesholger 449a705
fix _OPENMP version detect, including out-of-range for map version en…
jonesholger ed43d87
Merge branch 'develop' into pr-from-fork/137
rhornung67 592c854
More detail for the topdown/PAPI section of the documentation
jonesholger dfb6ad3
Update README.md
jonesholger ca71f90
Update README.md
jonesholger e8ade72
Update README.md
jonesholger 3b08adc
Merge branch 'use_caliper' into pr-from-fork/137
jonesholger fc5e6d3
update uberenv config file for Spack v0.19.0; add Caliper and Adiak p…
jonesholger 5499c7d
Merge branch 'develop' into pr-from-fork/137
rhornung67 a2708a1
Update (and fix) flags syntax in CI
adrienbernede e68710e
Update shared CI
adrienbernede a31768e
Attempt to build with caliper
adrienbernede 97edc52
Update tpl/RAJA to get latest changes in RAJA CI
adrienbernede 03d2562
Update extra specs in lassen CI
adrienbernede 49659db
Specify cuda_arch when building with Cuda
adrienbernede e5bdd6c
Fix caliper CI job name
adrienbernede 401e069
Merge pull request #280 from LLNL/woptim/caliper-integration
adrienbernede 54919cc
Add the option to not build and test in /dev/shm + copy caliper files…
adrienbernede b300358
Do not use /dev/shm for caliper job
adrienbernede fc2981a
Protect cali files copy with existence check
adrienbernede 3c6503e
Add caliper reference files + minor
adrienbernede 7731dc9
Minor
adrienbernede 256fc52
Add a comparison report between cali files using hatchet
adrienbernede 70b6cff
Fix script name and path
adrienbernede e3fc7d7
TEMP: de-activate unneeded part of the CI
adrienbernede ed6033a
Fix script call path
adrienbernede dbe10a4
Fix python script
adrienbernede aaa3361
Add missing extension
adrienbernede cb3742e
Fix baseline dir
adrienbernede 42f9619
Fix use existing field
adrienbernede e5ed98b
Revert "TEMP: de-activate unneeded part of the CI"
adrienbernede 5fa11bc
add support for cross-variant comparisons
jonesholger c46858e
replace optparse with argparse
jonesholger 09292db
Merge branch 'develop' into pr-from-fork/137
adrienbernede b134ad6
Merge branch 'pr-from-fork/137' into woptim/caliper-integration
adrienbernede 8fd43bd
Add usptream spack management to build_and_test script
adrienbernede 609549e
Fix typo
adrienbernede 6490c18
Protect against unititialized upstream
adrienbernede bd90e75
Fix typo
adrienbernede f010820
Prevent setting upstream when updating it
adrienbernede 6671991
Fix typo
adrienbernede 82c0423
Fix boolean use
adrienbernede 5969bf1
Merge branch 'develop' into pr-from-fork/137
adrienbernede e9b1ba0
Merge branch 'pr-from-fork/137' into woptim/caliper-integration
adrienbernede 3993588
Merge pull request #298 from LLNL/hatchet-analysis-crossvariant
adrienbernede b2ec163
Add some comments
adrienbernede aaba7f5
Deactivate a deprecated spec
adrienbernede bb52164
Remove reference files and move to in-commit perf comparison
adrienbernede 6b03dac
Store filtered graphframes in copy
adrienbernede e4ca868
Add caliper target to lassen and corona, add Cuda and OmpTarget analysis
adrienbernede 06ce8bc
Merge branch 'woptim/share-raja-perf-package' into pr-from-fork/137
adrienbernede 74a4f6e
Remove cuda-arch in non-cuda spec
adrienbernede f2dd795
Merge branch 'pr-from-fork/137' into woptim/caliper-integration
adrienbernede 489c9a5
Effectively use the upstream
adrienbernede 29dbcdb
Fix upstream path
adrienbernede b826e28
Remove superfluous comparison
adrienbernede c3a88ff
Merge branch 'develop' into pr-from-fork/137
adrienbernede cb65f40
Merge branch 'pr-from-fork/137' into woptim/caliper-integration
adrienbernede 7a1f7b9
Actually use Base implementation as baseline
adrienbernede bd0d585
Use new CI queue in RAJAPerf
adrienbernede 39e699d
Update Rad Spack Conf (via RAJA) to Add python and elfutils as extern…
adrienbernede 7306808
python 3.8.2 missing on toss4 machines, using 3.10.8
adrienbernede 2eac54f
No system elfutils on toss4
adrienbernede b7fbc8c
No system elfutils on blueos
adrienbernede 110d578
Point to RAJA@develop with new radiuss-spack-configs
adrienbernede b29d567
Merge pull request #291 from LLNL/woptim/caliper-integration
adrienbernede edaa2e2
Merge branch 'develop' into pr-from-fork/137
jonesholger 202635e
added sphinx doc for building with caliper, and Caliper output
jonesholger 548b42a
more generic MPI Init
jonesholger File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,7 @@ | ||
| *.pyc | ||
| *.o | ||
| *.exe | ||
| *.swp | ||
| *.swo | ||
| build*/ | ||
| install*/ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,40 @@ | ||
| #!/usr/bin/env bash | ||
|
|
||
| ############################################################################### | ||
| # Copyright (c) 2017-20, Lawrence Livermore National Security, LLC | ||
| # and RAJA Performance Suite project contributors. | ||
| # See the RAJAPerf/COPYRIGHT file for details. | ||
| # | ||
| # SPDX-License-Identifier: (BSD-3-Clause) | ||
| ################################################################################# | ||
|
|
||
| BUILD_SUFFIX=lc_blueos-nvcc10-caliper-gcc8.3.1 | ||
| RAJA_HOSTCONFIG=../tpl/RAJA/host-configs/lc-builds/blueos/nvcc_gcc_X.cmake | ||
|
|
||
| rm -rf build_${BUILD_SUFFIX} >/dev/null | ||
| mkdir build_${BUILD_SUFFIX} && cd build_${BUILD_SUFFIX} | ||
|
|
||
| module load cmake/3.14.5 | ||
| module load caliper-2.5.0-gcc-8.3.1-cu3vy3k | ||
|
|
||
| CALIPER_PREFIX=/usr/WS2/holger/spack/opt/spack/linux-rhel7-power9le/gcc-8.3.1/caliper-2.5.0-cu3vy3kjwjerpdm6xis2kauhz4s6wto2/ | ||
|
|
||
| ADIAK_PREFIX=/usr/WS2/holger/spack/opt/spack/linux-rhel7-power9le/gcc-8.3.1/adiak-0.2.1-hsv444o7ofb6s2znkvvnh6hcmr774g73/ | ||
|
|
||
| cmake \ | ||
| -DCMAKE_BUILD_TYPE=Release \ | ||
| -DCMAKE_CXX_COMPILER=/usr/tce/packages/gcc/gcc-8.3.1/bin/g++ \ | ||
| -C ${RAJA_HOSTCONFIG} \ | ||
| -DCMAKE_PREFIX_PATH="${CALIPER_PREFIX}/share/cmake/caliper;${ADIAK_PREFIX}/lib/cmake/adiak" \ | ||
| -DENABLE_OPENMP=On \ | ||
| -DENABLE_CUDA=On \ | ||
| -DCMAKE_CUDA_FLAGS="-Xcompiler -mno-float128" \ | ||
| -DCUDA_TOOLKIT_ROOT_DIR=/usr/tce/packages/cuda/cuda-10.2.89 \ | ||
| -DCMAKE_CUDA_COMPILER=/usr/tce/packages/cuda/cuda-10.1.243/bin/nvcc \ | ||
| -DCUDA_ARCH=sm_70 \ | ||
| -DENABLE_CALIPER=On \ | ||
| -DCMAKE_INSTALL_PREFIX=../install_${BUILD_SUFFIX} \ | ||
| -DCMAKE_EXPORT_COMPILE_COMMANDS=On \ | ||
| -DCMAKE_VERBOSE_MAKEFILE=On \ | ||
| "$@" \ | ||
| .. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this change here? Is this project-wide change necessary to support caliper? It does not seem to be related?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mcfadden8 Thanks for catching .gitmodules issue. I was probably advancing or editing BLT manually early on in the project. I'll clean it up