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

Backport/backport 211 to 1.x #239

Merged

Conversation

thanawan-atc
Copy link
Contributor

@thanawan-atc thanawan-atc commented Aug 22, 2023

Description

Backport PR #211

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.

dhrubo-os and others added 4 commits August 9, 2023 17:06
* updating notebook + bumping version

Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>

* addressing comments

Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>

---------

Signed-off-by: Dhrubo Saha <dhrubo@amazon.com>
(cherry picked from commit 1237aa6)
* Add description field

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Restore notebook

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Debug test

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Resolve linting issues

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Update CHANGELOG.md

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Debug test_sentencetransformermodel_pytest.py

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Improve test coverage

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Edit test name

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Change CHANGELOG.md & Add comment to sentencetransformermodel.py

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Correct linting

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Improve add description

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Correct linting

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Loosen restriction

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Update sentencetransformermodel.py

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Change function name + Add comment + Add default description

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

* Debug

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>

---------

Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>
(cherry picked from commit 20435b1)
Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>
Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>
Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>
Signed-off-by: Thanawan Atchariyachanvanit <latchari@amazon.com>
@codecov
Copy link

codecov bot commented Aug 22, 2023

Codecov Report

Merging #239 (2074326) into 1.x (5071bfb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              1.x     #239   +/-   ##
=======================================
  Coverage   91.30%   91.30%           
=======================================
  Files          38       38           
  Lines        4199     4199           
=======================================
  Hits         3834     3834           
  Misses        365      365           

@dhrubo-os dhrubo-os merged commit 8b3114d into opensearch-project:1.x Aug 22, 2023
13 checks passed
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.

3 participants