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.

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

Remove and reimplement remote variables to support CPU and GPU operations. #1

Merged
merged 72 commits into from
Aug 23, 2024

Conversation

Iainmon
Copy link
Owner

@Iainmon Iainmon commented Aug 23, 2024

This pull request is from a branch that was initially intended to remove all instances of the record remote(type eltType) for bare record ndarray(param rank: int, type eltType) data handling. This has been made to work and performance has been tuned and was used for my internship presentation (snapshot of this state here: cc67fb9).

Attached is my presentation: Iain_Moncrief_Summer_24_ChAI.pdf.

Since then, I have reimplemented remote variables, so that tensor operations can once again be done on the GPU. This was tested on an HPE Cray super computer with NVIDA GPUs. The data in the presentation was collected on the Horizon supercomputing system, (Cray XC).

@Iainmon Iainmon merged commit affbc30 into main Aug 23, 2024
3 checks passed
ShreyasKhandekar pushed a commit to ShreyasKhandekar/ChAI that referenced this pull request Sep 11, 2024
…gurable-dir

Make model directory configurable in loadFromSpec test to make integration with Chapel nightly testing easier.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant