-
Notifications
You must be signed in to change notification settings - Fork 19
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
Ray Integration to Stacks #52
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @elamaran11 for this stack. I will wait for the requested changes and then approve it.
ray-integration/README.md
Outdated
```bash | ||
idpbuilder create \ | ||
--use-path-routing \ | ||
--p https://github.com/cnoe-io/stacks//ray-integration |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The shorthand flag has to be -p
and longhand is --package
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
ray-integration/README.md
Outdated
```bash | ||
idpbuilder create \ | ||
--use-path-routing \ | ||
--p https://github.com/cnoe-io/stacks//ref-implementation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This command is missing multiline command format
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
@@ -0,0 +1,95 @@ | |||
# Make sure to increase resource requests and limits before using this example in production. | |||
# For examples with more realistic resource configuration, see | |||
# ray-cluster.complete.large.yaml and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we have references to these files?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No longer required, references are removed.
apiVersion: backstage.io/v1alpha1 | ||
kind: Location | ||
metadata: | ||
name: basic-example-templates | ||
description: A collection of example templates | ||
spec: | ||
targets: | ||
- ./ray-serve/template-ray-serve.yaml | ||
--- | ||
apiVersion: backstage.io/v1alpha1 | ||
kind: Location | ||
metadata: | ||
name: basic-organization | ||
description: Basic organization data | ||
spec: | ||
targets: | ||
- ./organization/guests.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should template-ray-serve.yaml
to : https://github.com/cnoe-io/stacks/tree/main/ref-implementation/backstage-templates/entities
OR
Remove basic-organization location and instructions to Register an existing component
using catalog-info.yaml from this stack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
|
||
train_config = { | ||
"lr": 1e-3, | ||
"epochs": 1, # artifically set low to finish quickly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: artifically ==> artificially
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done.
Signed-off-by: Elamaran Shanmugam <[email protected]>
Signed-off-by: Elamaran Shanmugam <[email protected]>
Signed-off-by: Elamaran Shanmugam <[email protected]>
@punkwalker All required changes and comments are incorporated. Please review and merge. |
This PR adds a new RAY Stack integration to Stacks. This PR includes the following: