-
Notifications
You must be signed in to change notification settings - Fork 7
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
[Documentation] Update Readme and Document using Destinations #231
Conversation
@@ -35,219 +31,101 @@ The SDK simplifies the setup and interaction with SAP AI Core, allowing you to f | |||
|
|||
To use the SDK in a Java application, it is necessary to understand the technical prerequisites and required versions for common dependencies. | |||
|
|||
- **SAP AI Core Service** enabled in your SAP BTP account. |
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.
Technically, this isn't necessary, only some form of access is needed.
|
||
Example Linux/MacOS: | ||
This quickstart assumes you have a **deployment of the Orchestration service available** in the `default` resource group of your AI Core instance. | ||
If you don't have a deployment yet, please refer to [this guide](https://help.sap.com/docs/sap-ai-core/sap-ai-core-service-guide/create-deployment-for-orchestration) on how to create one. |
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.
I intentionally didn't add the code to create the config + deployment using our SDK APIs. Doing that in Java code is somewhat tedious and has the danger that it is executed more than once, creating unnecessary / duplicate configurations / deployments.
But more strongly, this step will become obsolete soon (hopefully) with a default orchestration deployment being implemented.
|
||
```shell | ||
cf push |
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 section got removed, because it only works if the user has manually set up a CF app, which wasn't explained in this tutorial. We could also add it back in, but then I'd refer to our sample code manifest file as an example..
Context