Skip to content
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

Merged
merged 3 commits into from
Sep 18, 2024

Conversation

ruanyl
Copy link
Member

@ruanyl ruanyl commented Sep 13, 2024

  1. disable index patterns which data source has no agent configured for text to visualization
  2. extract vega json object from the response which is supposed to be in between tag

Description

[Describe what this change achieves]

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test.
  • New functionality has user manual doc added.
  • Commits are signed per the DCO using --signoff.

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.

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]>
@@ -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';
Copy link
Collaborator

Choose a reason for hiding this comment

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

new2?

Copy link
Member Author

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.

Copy link
Member Author

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!

Copy link
Collaborator

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

@ruanyl ruanyl merged commit 67e920c into opensearch-project:main Sep 18, 2024
9 checks passed
@ruanyl ruanyl added the backport 2.x Trigger the backport flow to 2.x label Sep 18, 2024
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 18, 2024
…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
ruanyl pushed a commit that referenced this pull request Sep 20, 2024
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Trigger the backport flow to 2.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants