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

Added GPU Support #923

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Arushi-Gupta13
Copy link

The modifications in this pull request allow for GPU computations through Metal so that a wider range of models become accessible for implementation onto different hardware environments. Integration of GPU devices to execute models in a possibly more efficient way without any compromise to existing functionality is the primary focus of the changes.

Highlights:
Metal GPU Support: The model was rewritten in order to have Metal-enabled GPU execution capability for carrying out computations more efficiently on supported hardware.
Device Compatibility: Enhanced compatibility checks were implemented to ensure the model can properly run on the CPU and GPU, whereas, in the case of any device mismatch, appropriate error handling is set up.
Error Handling: A strong error-handling mechanism was laid down where each petty error is caught and returned when there is an attempt to use a model on an incompatible device (for instance, using GPU with CPU data or vice versa).
Tests for GPU/CPU Mismatches: Far-reaching tests were implemented to check that the model behaves properly on both CPU and GPU and to check that errors are raised when there is a mismatch between the model and the data device.

Testing:
Device Tests: Extensive tests were carried out to check the model's functionality on both CPU and GPU, as well as test beyond that for the right handling of device mismatches.
Scalar Input Handling: Tests were added to ensure that scalar inputs are treated correctly for both the CPU and GPU.

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