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 main] adding model level metric in node level #1522

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport bb84282 from #1330

* adding model level metric in node level

Signed-off-by: Dhrubo Saha <[email protected]>

* spotlessApply and fixed a test

Signed-off-by: Dhrubo Saha <[email protected]>

* added if clause to bypass the integration test

Signed-off-by: Dhrubo Saha <[email protected]>

* addressing comments

Signed-off-by: Dhrubo Saha <[email protected]>

* addressed comments

Signed-off-by: Dhrubo Saha <[email protected]>

* add more tests

Signed-off-by: Dhrubo Saha <[email protected]>

* adding boolean check if model stats exists

Signed-off-by: Dhrubo Saha <[email protected]>

---------

Signed-off-by: Dhrubo Saha <[email protected]>
(cherry picked from commit bb84282)
@opensearch-trigger-bot opensearch-trigger-bot bot temporarily deployed to ml-commons-cicd-env October 16, 2023 22:44 Inactive
@opensearch-trigger-bot opensearch-trigger-bot bot temporarily deployed to ml-commons-cicd-env October 16, 2023 22:44 Inactive
@opensearch-trigger-bot opensearch-trigger-bot bot temporarily deployed to ml-commons-cicd-env October 16, 2023 22:44 Inactive
@ylwu-amzn ylwu-amzn merged commit 701af31 into main Oct 16, 2023
6 of 9 checks passed
@github-actions github-actions bot deleted the backport/backport-1330-to-main branch October 16, 2023 23:04
TrungBui59 pushed a commit to TrungBui59/ml-commons that referenced this pull request Nov 21, 2023
…ensearch-project#1522)

* adding model level metric in node level

Signed-off-by: Dhrubo Saha <[email protected]>

* spotlessApply and fixed a test

Signed-off-by: Dhrubo Saha <[email protected]>

* added if clause to bypass the integration test

Signed-off-by: Dhrubo Saha <[email protected]>

* addressing comments

Signed-off-by: Dhrubo Saha <[email protected]>

* addressed comments

Signed-off-by: Dhrubo Saha <[email protected]>

* add more tests

Signed-off-by: Dhrubo Saha <[email protected]>

* adding boolean check if model stats exists

Signed-off-by: Dhrubo Saha <[email protected]>

---------

Signed-off-by: Dhrubo Saha <[email protected]>
(cherry picked from commit bb84282)

Co-authored-by: Dhrubo Saha <[email protected]>
Signed-off-by: TrungBui59 <[email protected]>
austintlee pushed a commit to austintlee/ml-commons that referenced this pull request Feb 29, 2024
…ensearch-project#1522)

* adding model level metric in node level

Signed-off-by: Dhrubo Saha <[email protected]>

* spotlessApply and fixed a test

Signed-off-by: Dhrubo Saha <[email protected]>

* added if clause to bypass the integration test

Signed-off-by: Dhrubo Saha <[email protected]>

* addressing comments

Signed-off-by: Dhrubo Saha <[email protected]>

* addressed comments

Signed-off-by: Dhrubo Saha <[email protected]>

* add more tests

Signed-off-by: Dhrubo Saha <[email protected]>

* adding boolean check if model stats exists

Signed-off-by: Dhrubo Saha <[email protected]>

---------

Signed-off-by: Dhrubo Saha <[email protected]>
(cherry picked from commit bb84282)

Co-authored-by: Dhrubo Saha <[email protected]>
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.

2 participants