-
Notifications
You must be signed in to change notification settings - Fork 138
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
adds deployed prompt endpoint functionality #249
base: master
Are you sure you want to change the base?
adds deployed prompt endpoint functionality #249
Conversation
@jwm4 fyi |
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 actions json does not work as intended on load in to Watson X assistant. For deployed models in WatsonX AI.
Variable is not correctly set and thus the condition that relies on the variable always fails.
"skill_variable": "model_parameters_max_new_tokens" | ||
}, | ||
"parameter": "parameters.max_new_tokens" | ||
"variable": "step_817_result_2", |
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.
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 haven't tried it out yet, but that typically happens when the actions.json file was exported from an older version of the openapi spec. So @modcarroll , I would recommend that you set this up again with the latest openapi spec and then re-do the export of the actions.json file.
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 believe I fixed this issue - can you check the latest commit and check again?
...deployments/{deployment-id}/generation/text
endpoing to the openAPI file in order to use deployed prompts