Remove dead CuPy link, refresh Xena/K40 GPU benchmarks with real Easley data#258
Closed
Graviton28 wants to merge 3 commits into
Closed
Remove dead CuPy link, refresh Xena/K40 GPU benchmarks with real Easley data#258Graviton28 wants to merge 3 commits into
Graviton28 wants to merge 3 commits into
Conversation
Fixes #151. The referenced file was never actually added to the repo (checked full history and all branches — no trace of it anywhere), so the README link has been a 404 since it was written. Removing rather than recreating content that never existed.
Fixes #235. The old "CARC Benchmarks" section referenced an NVIDIA Tesla K40 on the retired Xena cluster. Removed it entirely and replaced it with real benchmark data + fresh plots generated on Easley's l40s partition (single L40S GPU), comparing CPU vs GPU for matrix multiplication and inversion at N=1000/2000/4000/8000.
The CuPy dead link is being replaced with a real doc (cupy_intro.md, separate PR) rather than just deleted, so this PR shouldn't touch that line anymore - just the benchmark refresh (#235).
This was referenced Jul 7, 2026
Contributor
Author
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
CuPy_on_CARC.ipynb, which was never actually added to the repo (checked full history/all branches — no trace of it anywhere).multiGPU_tensorflow_tutorial.md, which referenced an NVIDIA Tesla K40 on the retired Xena cluster, and replaces it with fresh benchmark data and plots generated on Easley'sl40spartition (single L40S GPU) comparing CPU vs GPU for matrix multiplication and inversion at N=1000/2000/4000/8000.Test plan
CuPy_on_CARC.ipynbexists anywhere in repo history or branches.Images/matrix_inverse.png,Images/matrix_multiplication.png) aren't referenced by any other doc before replacing them.