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 support for g3s.xlarge and g4dn.xlarge instances #140

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

acap4z
Copy link

@acap4z acap4z commented Jan 29, 2021

Added support for g3s.xlarge and g4dn.xlarge instances by changing base AMI to "Deep Learning Base AMI (Amazon Linux 2)". This one has Nvidia drivers and CUDA support so it seems to meet the requirements to be used in NPK. It has the advantage of supporting a wider range of instance types, including the newest G4 and P4. G4 has been benchmarked and shows the best results in terms of performance/$.

Changing to this AMI may solve some other limitations such as the one explained in #50 related to regions.

imagen

imagen

Added g3s.xlarge instance support by changing base AMI to the Deep Learning Base.
Added g4dn.xlarge instance support. Benchmarks show that it is the most efficient GPU instance available right now.
@acap4z acap4z changed the title Added g3s.xlarge instance support Added support for g3s.xlarge and g4dn.xlarge instances Feb 5, 2021
@c6fc
Copy link
Contributor

c6fc commented Feb 5, 2021

Interesting idea. I'll test this in dev when I get a chance. There are many more places where new instances types have to be configured in order for them to function correctly, but I think it's a valid thing to add.

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.

2 participants