Skip to content
This repository was archived by the owner on Apr 8, 2024. It is now read-only.

[hold] Add batch latency measurement and percentiles to treelite_python - #177

Open
Jeff Omhover (jfomhover) wants to merge 19 commits into
mainfrom
jfomhover/treelitepercentiles
Open

[hold] Add batch latency measurement and percentiles to treelite_python#177
Jeff Omhover (jfomhover) wants to merge 19 commits into
mainfrom
jfomhover/treelitepercentiles

Conversation

@jfomhover

@jfomhover Jeff Omhover (jfomhover) commented Nov 24, 2021

Copy link
Copy Markdown
Contributor

This PR introduces num_threads and batch_size as parameters for inferencing benchmark.

Additions:

  • code to process batches of data in the treelite_python module,
  • unified code to record latency metrics and percentiles.
  • num_threads and batch_size in inferencing variant parameters
  • num_threads and batch_size in the inferencing pipeline (depending on support from each component)

@jfomhover Jeff Omhover (jfomhover) added the enhancement New feature or request label Nov 24, 2021
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 07:08 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 07:11 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 07:14 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 17:07 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:22 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:23 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:31 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:44 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:46 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 18:49 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 24, 2021 19:03 Inactive
@github-actions

github-actions Bot commented Nov 24, 2021

Copy link
Copy Markdown

Unit Test Results for Build

  1 files  ±0    1 suites  ±0   7s ⏱️ ±0s
76 tests ±0  76 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 51c375b. ± Comparison against base commit 7b8a339.

♻️ This comment has been updated with latest results.

@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops November 25, 2021 05:12 Inactive
@jfomhover Jeff Omhover (jfomhover) changed the title Add batch latency measurement and percentiles to treelite_python [hold] Add batch latency measurement and percentiles to treelite_python Nov 29, 2021
@jfomhover Jeff Omhover (jfomhover) added the hold This PR/Issue should be put on hold. label Nov 29, 2021
@jfomhover

Copy link
Copy Markdown
Contributor Author

Split changes into a PR to unify metrics first, waiting for #180 merged

@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops December 3, 2021 19:00 Inactive
@jfomhover
Jeff Omhover (jfomhover) temporarily deployed to mlops December 3, 2021 19:02 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request hold This PR/Issue should be put on hold.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant