Skip to content

docs(python): rpt 1.6 updated - #832

Open
yamaceay wants to merge 2 commits into
mainfrom
rpt-clients
Open

yamaceay wants to merge 2 commits into
mainfrom
rpt-clients

Conversation

@yamaceay

Copy link
Copy Markdown
Contributor

What Has Changed?

The latest PR (SAP/ai-sdk-python#174) which enables the most recent RPT clients (1.5 and 1.6) has been successfully merged in SAP/ai-sdk-python, and docs have to be updated minimally to reflect the current changes. This PR replaces sap-rpt-1-small by sap-rpt-1.6 and adds prediction_placeholder parameter to the TargetColumn, which is now required from v1.5 on.

@yamaceay
yamaceay requested a review from mwien October 20, 2026 08:58

@mwien mwien left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few small changes regarding how we mention the version/model numbers.

Comment thread docs-python/examples/gen-ai-hub.mdx Outdated

You can get predictions directly by the native SDK client.

For detailed information visit the [SAP Help Portal](https://help.sap.com/docs/sap-ai-core/generative-ai/example-payloads-for-inferencing-sap-rpt-1?locale=en-US).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link either to RPT 1.6 or to the overall RPT section.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://help.sap.com/docs/sap-ai-core/generative-ai/sap-rpt-1?locale=en-US

We cannot link to the overall RPT section, so linking directly to 1.6 would be the other option, but it only introduces explainability so I fear we should look for another third option.

Comment thread docs-python/examples/gen-ai-hub.mdx Outdated
Comment thread docs-python/examples/gen-ai-hub.mdx Outdated

@OscarDDD OscarDDD left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[pp] Since Java and Javascript has a separate page for SAP RPT. Would it make sense for us to just move this to a separate page ? Ofc, we can make a new ticket for this if necessary.

@mwien

mwien commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

[pp] Since Java and Javascript has a separate page for SAP RPT. Would it make sense for us to just move this to a separate page ?

Yes agreed, but I'd prefer tp keep it out of this ticket, which is only about the RPT update to 1.6.

OscarDDD
OscarDDD previously approved these changes Oct 26, 2026

This branch has not been deployed

No deployments
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