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

Config - Add T4 configurations with inference benchmark #311

Merged
merged 2 commits into from
Feb 20, 2022
Merged

Config - Add T4 configurations with inference benchmark #311

merged 2 commits into from
Feb 20, 2022

Conversation

abuccts
Copy link
Member

@abuccts abuccts commented Feb 15, 2022 •
edited by cp5555
Loading

Add T4 configurations with inference benchmark.

@abuccts abuccts added the configuration Benchmark configurations label Feb 15, 2022
@abuccts abuccts requested a review from a team as a code owner February 15, 2022 08:58
@codecov
Copy link

codecov bot commented Feb 15, 2022 •
edited
Loading

Codecov Report

Merging #311 (f906a0b) into main (97ed12f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #311   +/-   ##
=======================================
  Coverage   88.64%   88.64%           
=======================================
  Files          76       76           
  Lines        4500     4500           
=======================================
  Hits         3989     3989           
  Misses        511      511
Flag Coverage Δ
cpu-unit-test 72.75% <ø> (ø)
cuda-unit-test 88.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 97ed12f...f906a0b. Read the comment docs.

@abuccts abuccts force-pushed the xiongyf/inference-config branch 3 times, most recently from a6c861c to 4582b09 Compare February 15, 2022 11:30
@abuccts
Add configurations for inference
866e5c9 
Add configurations for inference.
@cp5555 cp5555 changed the title Config - Add configurations for inference Config - Add T4 configurations for inference Feb 19, 2022
@cp5555 cp5555 changed the title Config - Add T4 configurations for inference Config - Add T4 configurations with inference benchmarking Feb 19, 2022
@cp5555 cp5555 mentioned this pull request Feb 19, 2022
Closed
@cp5555 cp5555 changed the title Config - Add T4 configurations with inference benchmarking Config - Add T4 configurations with inference benchmark Feb 19, 2022
@cp5555 cp5555 self-assigned this Feb 20, 2022
@abuccts abuccts enabled auto-merge (squash) February 20, 2022 12:40
@abuccts abuccts merged commit ea2c10a into main Feb 20, 2022
@abuccts abuccts deleted the xiongyf/inference-config branch February 20, 2022 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yukirora yukirora yukirora approved these changes

@cp5555 cp5555 cp5555 approved these changes

@guoshzhao guoshzhao Awaiting requested review from guoshzhao

Assignees

@cp5555 cp5555

Labels
configuration Benchmark configurations
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abuccts @cp5555 @yukirora