-
Notifications
You must be signed in to change notification settings - Fork 17
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
Spatter Refactor - C++, MPI, Parsing, CMake updates #165
Conversation
…be changed on command line
@plavin Could you take a look at the CUDA kernels when you get the chance? I'll post updated performance number below and if things look okay to you we can try to port this over to the Spatter My work around to get everything working with
I've ran this in 2 configurations for the gather:
Note that Spatter original uses configuration 2. |
GPU Performance still needs some work (this is a V100)
Update (#154) : @plavin @jyoung3131
Configuration 1: ![]() Configuration 2: ![]() |
Fixes Pattern Parsing
Update Tests
Fix Tests for Serial and OpenMP Backends
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.
We've reviewed the codebase, tested multiple variations, and reviewed all the outstanding issues. While there are likely some small bugfixes and additional testing, we've pulled these out into smaller issues that can be addressed, as needed.
Re-factoring to C++ to simplify parsing and memory management. The goal is to be as backwards-compatible as possible, from command line flags and behavior, to performance and output.
Currently, the only difference in command line flags is the usage of
-u <--pattern-scatter>
rather than-h <--pattern-scatter>
to free up the-h
flag for--help
and the-q <--no-print-header>
flag's functionality has been moved to-v <--verbosity>
. We will have to find a solution for the previous-v <--vector-len>
flag. Finally, the-f <--kernel-file>
flag is now used to direct to the JSON file, rather than overloading the-p
flag.Changes:
--help
-->-h <--help>
-h <--pattern-scatter>
-->-u <--pattern-scatter>
-q <--no-print-header>
-->-v <--verbose>
-v <--vector-len>
--> ?-f <--kernel-file>
-->-f <--json-file>
Are we interested in completing/pursuing this?
Complete:
To-do:
- [ ] Multiple Deltas Capability- [ ] OpenCL Support- [ ] Op Support- [ ] PAPI Support- [ ] RO_Hilbert, RO_Morton, Strided Support- [ ] vector-len Support- [ ] Validation moved to GPU Validation and Readd Validate Flag #194Future releases:
Not Needed:
- Old utility files (trace_util.c for example)