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

[Feature] Enable ppl visualization in Chatbot #1304

Conversation

SuZhou-Joe
Copy link
Member

@SuZhou-Joe SuZhou-Joe commented Dec 18, 2023

Description

Enable ppl_visualization with Chatbot.

image

image

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass, including unit test, integration test and doctest
  • New functionality has been documented.
    • New functionality has javadoc added
    • 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.

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@SuZhou-Joe SuZhou-Joe changed the title [Feature] Enable ppl visualization on Chatbot [Feature] Enable ppl visualization in Chatbot Dec 18, 2023
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@SuZhou-Joe SuZhou-Joe force-pushed the feature/enable_ppl_visualization_on_main branch from 37c5ec6 to 51ab4aa Compare December 18, 2023 02:55
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@SuZhou-Joe
Copy link
Member Author

SuZhou-Joe commented Dec 18, 2023

@joshuali925 @anirudha Would you mind taking a look on this PR?

Copy link
Member

@joshuali925 joshuali925 left a comment

Choose a reason for hiding this comment

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

minor comment, thanks

* It will gives an type error when dashboards-assistant is not installed so add a ts-ignore to suppress the error.
*/
// @ts-ignore
export type { AssistantSetup } from "../../dashboards-assistant/public";
Copy link
Member

Choose a reason for hiding this comment

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

also add this to

export interface SetupDependencies {
embeddable: EmbeddableSetup;
visualizations: VisualizationsSetup;
data: DataPublicPluginSetup;
uiActions: UiActionsStart;
managementOverview?: ManagementOverViewPluginSetup;
}
?
assistantDashboards?: AssistantSetup;

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure, done for this.

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@SuZhou-Joe
Copy link
Member Author

One more approval is needed, can anyone help to review please?

@joshuali925
Copy link
Member

maybe @paulstn? could you help to review

@joshuali925 joshuali925 merged commit 1110e0c into opensearch-project:main Dec 21, 2023
8 of 15 checks passed
@SuZhou-Joe
Copy link
Member Author

@joshuali925 , could you please add backport 2.x label?

opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 19, 2024
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
(cherry picked from commit 1110e0c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
joshuali925 pushed a commit that referenced this pull request Jan 23, 2024
(cherry picked from commit 1110e0c)

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
RyanL1997 pushed a commit to RyanL1997/dashboards-observability that referenced this pull request Apr 18, 2024
…roject#1342)

The dashboard has references to an old path that does not exist anymore:
`plugins/opensearch-security/securityconfig/config.yml`.  It was
replaced by `config/opensearch-security/config.yml` in this commit:

opensearch-project/security@b44d7ea

Fix the last occurences of `plugins/opensearch-security/securityconfig`.

Signed-off-by: Romain Tartière <romain@blogreen.org>
Co-authored-by: Stephen Crawford <65832608+scrawfor99@users.noreply.github.com>
(cherry picked from commit 82c27c7c235a3f075e697d566b6fe3c97f4fdcbd)

Co-authored-by: Romain Tartière <romain@blogreen.org>
Co-authored-by: Craig Perkins <cwperx@amazon.com>
amsiglan pushed a commit to amsiglan/dashboards-observability that referenced this pull request Jun 7, 2024
… (opensearch-project#1374)

(cherry picked from commit 1110e0c)

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit d9bd287)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants