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

add ray cluster launcher files for aws, azure, and gcp #305

Open
wants to merge 3 commits into
base: ray_cluster_functionality
Choose a base branch
from

Conversation

nanli-emory
Copy link
Collaborator

No description provided.

@choosehappy
Copy link
Owner

Thanks Nan. can you describe what tests you performed and what the results of those tests were? are these the default config files? did you have to make any modifications to get things working?

@nanli-emory
Copy link
Collaborator Author

Thanks Nan. can you describe what tests you performed and what the results of those tests were? are these the default config files? did you have to make any modifications to get things working?

Hi @choosehappy and @jacksonjacobs1, I added a Readme.md under the folder as an instruction. Please review it.

@choosehappy
Copy link
Owner

looks pretty darn cool, thanks Nan! next step here i think is for @jacksonjacobs1 to run through the commands provided and see if he can trivially recapitulate, and if so, I think we can call this first round of tests successful. We'll need to further sketch out the design documents to see exactly if there are other tests that should be performed, but not sure if any of them would be impacted by this work -- essentially once ray is "up" and running, it should be sufficiently abstracted to use in code that everything should work (i.e., this is the main purpose of ray). a load off my mind : )

@jacksonjacobs1
Copy link
Collaborator

Thanks Nan! I'll test this out today and will follow up.

@nanli-emory
Copy link
Collaborator Author

Thanks Nan! I'll test this out today and will follow up.
Hi @jacksonjacobs1,
Please DM me on Teams if you have any issues.

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.

3 participants