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

Fix assistant failed cases because of change of ml-commons #1109

Closed
wants to merge 10 commits into from

Conversation

SuZhou-Joe
Copy link
Member

@SuZhou-Joe SuZhou-Joe commented Feb 18, 2024

Description

ml-commons changes the prompt when using tool in agent framework

Issues Resolved

[List any issues this PR will resolve]

Check List

  • 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.

opensearch-trigger-bot bot and others added 10 commits February 6, 2024 19:47
opensearch-project#1075)

Signed-off-by: Junqiu Lei <junqiu@amazon.com>
(cherry picked from commit 09f9f3e)

Co-authored-by: Junqiu Lei <junqiu@amazon.com>
…oject#1074) (opensearch-project#1078)

* allow switch table
* make sure the DatePickerMenu is open
* update switchDiscoverTable to accomadate changing from toggle to button
* skip saved queries
* fixed flaky tests and missing ones

---------

Signed-off-by: Anan Zhuang <ananzh@amazon.com>
(cherry picked from commit 34a2630)

Co-authored-by: Anan Zhuang <ananzh@amazon.com>
* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated intercepts

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* fixed lint errors

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated tests

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* fixed incorrect intercept setup

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

* updated rules test

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>

---------

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
* Fix in-context report tests

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Switch to checking for success toast

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Fix syntax

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

---------

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
Adds data test sub

Signed-off-by: Rupal Mahajan <maharup@amazon.com>
* increase delay times in gantt charts

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
(cherry picked from commit 508fd25)

* update create button text

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

* revert create button change

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>

---------

Signed-off-by: Shenoy Pratik <sgguruda@amazon.com>
Signed-off-by: Paul Sebastian <paulstn@amazon.com>
…d error (opensearch-project#1102)

Signed-off-by: Amardeepsingh Siglani <amardeep7194@gmail.com>
* Normalize string newlines

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Revert "Normalize string newlines"

This reverts commit f25a99f.

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

* Fix which side is newline-normalized

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>

---------

Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants