chore: Update model list #591
Conversation
.changeset/shy-socks-shake.md
Outdated
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| '@sap-ai-sdk/core': patch | |||
There was a problem hiding this comment.
This should be a minor release in my opinion.
There was a problem hiding this comment.
I would say from the users perspective, the impacted packages are orchestration and foundation-models.
There was a problem hiding this comment.
I changed it to minor.
Regarding the second point, I am still in favour of noting the changes as part of the core package. As almost all functionality in core is meant to be used by other packages, it will basically ask us to never put core in release note. And some changes could affect very widely, such as introducing ErrorWithCause in very low level in core. In such case, I prefer not to put all other packages here except core, even if all other packages has zero code changes.
There was a problem hiding this comment.
Sorry for the confusion, I didn't imply removing core from affected packages in the changeset, rather also including the other 2.
Since we say core is internal and not for users, the changelog might imply that the change is restricted to core. However, the main change is seen in the orchestration/foundation-models by the users. The release notes are mainly for public api changes.
Context
Closes SAP/ai-sdk-js-backlog#244.
What this PR does and why it is needed
Add
o1,o3-miniandalephalpha-pharia-1-7b-controlto the available model list.