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.

Sign up for GitHub

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

Jump to bottom

Feature - Run pod #332

Merged
merged 25 commits into from
Feb 4, 2025
Merged

Feature - Run pod #332

merged 25 commits into from
Feb 4, 2025

Conversation

shincap8
Copy link
Contributor

@shincap8 shincap8 commented Feb 4, 2025

Context

  • There wasn't a proper log to identify which email after heavy evaluation was being sent (error or success)

Changes Made

  1. Include prints after sending the email.
  • This would allow the development team to properly Identify which email was being sent and to whom.

shincap8 and others added 25 commits January 16, 2025 19:25
@shincap8
Update model methods to run background task when creating the endpoin…
04cd514 
…t and creating score endpoint for runpod to use
@shincap8
Handle error from Runpod, URL inference from yaml file
d86ae16
@shincap8 @remg1997
Update backend/app/api/endpoints/base/score.py
8794693 
Co-authored-by: Rafael Mosquera <[email protected]>
@shincap8
Rename method
fc08634
@shincap8
Merge with cloud branch
aa0f022
@shincap8
rename endpoint
971b986
@shincap8
Print background tasks and correct typo
476d4e4
@shincap8
resolve merge conflict
9ed5f20
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
326acbb 
…ature/RunPod
@shincap8
Correct typos and correct schemas
68f126e
@shincap8
model uid
c2638b9
@shincap8
merge with main
605d8fb
@shincap8
remove c
0d74498
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
9c6f088 
…ature/RunPod
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
ac0b90b 
…ature/RunPod
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
22fe4f2 
…ature/RunPod
@shincap8
ad ta to metada (facepalm)
6afb9b1
@shincap8
add update publish status to make change the publish state of the models
a9fea53
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
5bb2a9b 
…ature/RunPod
@shincap8
change order of background tasks
72a44d6
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
a314b98 
…ature/RunPod
@shincap8
print instead of raising error for the second background task to run
255ea26
@shincap8
Merge branch 'main' of https://github.com/mlcommons/dynabench into Fe…
34ef670 
…ature/RunPod
@shincap8
remove update publish status from pipeline
659b59d
@shincap8
implement better logging when email send in heavy inference pipeline
b50ce40
@shincap8 shincap8 requested a review from a team as a code owner February 4, 2025 17:01
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 4, 2025

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

@shincap8 shincap8 merged commit e54520d into main Feb 4, 2025
13 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2025
@shincap8 shincap8 deleted the Feature/RunPod branch February 4, 2025 19:02
@shincap8 shincap8 restored the Feature/RunPod branch February 4, 2025 19:42
@shincap8 shincap8 deleted the Feature/RunPod branch February 4, 2025 19:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@remg1997 remg1997 remg1997 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@shincap8 @remg1997