-
Notifications
You must be signed in to change notification settings - Fork 23
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
disable index patterns which data source has no agent configured for text to visualization #310
disable index patterns which data source has no agent configured for text to visualization #310
Conversation
text to visualization + extract vega json object from the response which is supposed to be in between tag <vega-lite></vega-lite> Signed-off-by: Yulong Ruan <[email protected]>
common/constants/llm.ts
Outdated
@@ -43,3 +43,6 @@ export const NOTEBOOK_API = { | |||
export const DEFAULT_USER_NAME = 'User'; | |||
|
|||
export const TEXT2VEGA_INPUT_SIZE_LIMIT = 400; | |||
|
|||
export const TEXT2VEGA_AGENT_CONFIG_ID = 'os_text2vega_new2'; |
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.
new2?
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.
My bad, need to clean that up, it's for testing.
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.
@Hailong-am This is fixed now, could you please take another look? Thanks!
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, it looks good to me
Signed-off-by: Yulong Ruan <[email protected]>
Signed-off-by: Yulong Ruan <[email protected]>
…text to visualization (#310) * disable index patterns which data source has no agent configured for text to visualization + extract vega json object from the response which is supposed to be in between tag <vega-lite></vega-lite> Signed-off-by: Yulong Ruan <[email protected]> * text2viz only support opensearch type of data source Signed-off-by: Yulong Ruan <[email protected]> * update changelogs Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> (cherry picked from commit 67e920c) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md
…text to visualization (#310) (#314) * disable index patterns which data source has no agent configured for text to visualization + extract vega json object from the response which is supposed to be in between tag <vega-lite></vega-lite> Signed-off-by: Yulong Ruan <[email protected]> * text2viz only support opensearch type of data source Signed-off-by: Yulong Ruan <[email protected]> * update changelogs Signed-off-by: Yulong Ruan <[email protected]> --------- Signed-off-by: Yulong Ruan <[email protected]> (cherry picked from commit 67e920c) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.