-
Notifications
You must be signed in to change notification settings - Fork 106
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
CUDA & compression #267
Merged
Merged
CUDA & compression #267
Conversation
This file contains 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
harold-b
force-pushed
the
cuda-compression
branch
from
February 10, 2023 06:30
d60b1da
to
a852839
Compare
wallentx
previously requested changes
Mar 5, 2023
- Add power simulation mode via `--power`. - Fix small bug in decompression which used more compute than necessarry.
to chiapos's but with random seeds.
And other simulator fixes
* Added support for first pass of CUDA-based harvesting, with low matching performance. * Added support for C9 plots * Disabled creation of C8 plots for now (compression values not yet generated) * PlotWriter consecutive plot bug workaround for `cudaplot` * PlotWriter consecutive plot bug workaround for `ramplot`
* Refactor CMake build files * Created separate `bladebit_harvester` target meant to be used as a dependency. * Spelling fixes and build fixes. * Fixed CUDA harvesting qualities * Updated CI to for new harvester target * Other bug fixed * Made harvester API more portable, and able to be dynamically loaded * Add API to get compression info directly into the harvester API * version bump alpha4 Co-authored-by: Kyle Altendorf <k.altendorf@chia.net> Co-authored-by: William Allen <w.allen@chia.net>
* Adding harvester build changes * Enabled windows harvester builds w/ CUDA --------- Co-authored-by: Harold Brenes <h.brenes@chia.net>
harold-b
force-pushed
the
cuda-compression
branch
from
May 12, 2023 21:42
400180c
to
85c7654
Compare
Addresses issue reported in #319
* adding gencode fix for linker * I AM LINTER * argg keyboard
* Fixing arm64 conditional * adding arm64 builds * Remove else
* Testing m1 mac builds * Adding libsodium * export cpath and library path * Building libsodium from src * fixing syntax * Specify host * Adding paths Signed-off-by: wallentx <william.allentx@gmail.com> * Fixing CI Signed-off-by: wallentx <william.allentx@gmail.com> * Fix conditional Signed-off-by: wallentx <william.allentx@gmail.com> * Re-enabling all other jobs Signed-off-by: wallentx <william.allentx@gmail.com> * Make libsodium optional --------- Signed-off-by: wallentx <william.allentx@gmail.com>
Fix for gcc13
- Add in CPU-only macOS harvesting at the build level. - GreenReaper DLL target also available as a separate one from the static one.
Update FindCatch2.cmake
* Update CMakeLists.txt tracking of bls-signatures
hoffmang9
approved these changes
Jun 26, 2023
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.
lgtm
* Fixing issue with CUDA decompression context remaining unusable after an error. * Fix virtual memory allocation issues. * Implement missing SysHost::VirtualProtect on Windows. * Revert harvester to single, configurable lib target, defaulting to a dynamic library.
harold-b
force-pushed
the
cuda-compression
branch
from
July 2, 2023 20:35
0ee2a1e
to
aee1b64
Compare
in the GPU decompressor
harold-b
force-pushed
the
cuda-compression
branch
from
July 14, 2023 06:46
6e60063
to
1e5eb6e
Compare
* Enable PDB-based stack trace dumping on windows * Dump stack trace on VirtualFree failure
* Shrinking cuda install, add all assets to attach * Bumping runner image * Adding redhat-lsb-core * Trying local install without samples * Trying our manylinux cuda image * More cleanup * toolkit only * Set version for RC * Fixing artifact extension nesting * Fix windows artifact extension
* Add CentOS arm64 CI for bladebit * fix centos workflow
Signed-off-by: wallentx <william.allentx@gmail.com>
Adding gencode for Orin
* passing all architectures explicitly into bladebit harvester library
* Adding cuobjdump step
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.