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

[EAGLE-3792]-Test case for model upload #256

Merged
merged 7 commits into from
Jan 10, 2024
Merged

[EAGLE-3792]-Test case for model upload #256

merged 7 commits into from
Jan 10, 2024

Conversation

phatvo9
Copy link
Contributor

@phatvo9 phatvo9 commented Dec 28, 2023

Test dummy models for triton code


steps:
- name: Install dependencies
run: |
Copy link
Contributor

Choose a reason for hiding this comment

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

the indentation is broken here, you either need to indent lines 17-20 to match 16 or remove 2 spaces from 16 to match 17-20 and then also fix line 22 and below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for spotting that error

- name: start triton
run: |
tritonserver --model-repository ./tests/dummy_triton_models --exit-on-error false &
background: true
Copy link
Contributor

Choose a reason for hiding this comment

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

this doesnt seem to actually do anything

.github/workflows/run_triton_test.yaml Outdated Show resolved Hide resolved
.github/workflows/run_triton_test.yaml Outdated Show resolved Hide resolved
@ackizilkale
Copy link
Contributor

This is necessarily a huge PR. Went through it, did not find anything wrong.

@phatvo9 how long do you think these tests will take to run?

@phatvo9
Copy link
Contributor Author

phatvo9 commented Dec 29, 2023

This is necessarily a huge PR. Went through it, did not find anything wrong.

@phatvo9 how long do you think these tests will take to run?

With these dummy models, I think the test won't take more than 15 seconds

@phatvo9 phatvo9 requested a review from travertischio January 2, 2024 15:45
Copy link
Contributor

@ackizilkale ackizilkale left a comment

Choose a reason for hiding this comment

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

lgtm

@phatvo9 phatvo9 merged commit b06bb30 into master Jan 10, 2024
18 checks passed
@phatvo9 phatvo9 deleted the EAGLE-3792 branch January 10, 2024 04:46
@sanjaychelliah sanjaychelliah mentioned this pull request Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants