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

SharedProcessPool benchmarking #1904

Closed
wants to merge 30 commits into from
Closed

SharedProcessPool benchmarking #1904

wants to merge 30 commits into from

Conversation

yczhang-nv
Copy link
Contributor

@yczhang-nv yczhang-nv commented Sep 20, 2024 •
edited
Loading

Description

Code for a quick benchmarking for SharedProcessPool. Will be closed after the benchmark finished.

By Submitting this PR I confirm:

  • I am familiar with the Contributing Guidelines.
  • When the PR is ready for review, new or existing tests cover these changes.
  • When the PR is ready for review, the documentation is up to date with these changes.

@yczhang-nv
initial commit for shared_process_pool
7951324
@yczhang-nv
fix CI
f8037af
@yczhang-nv
Merge remote-tracking branch 'upstream/branch-24.10' into impl-multi-…
6a48cf3 
…process-stage
@yczhang-nv
Add performance verification tests to shared_process_pool
1443d11
@yczhang-nv
Add tests and fix shutdown of shared_process_pool
86cbed8
@yczhang-nv
Initial impl of multi_processing_stage
21fd8e8
@yczhang-nv
initial test for test_multi_processing_stage
46865ae
@yczhang-nv
fix unit test for multi_processing_stage
406dbeb
@yczhang-nv
Merge remote-tracking branch 'upstream/branch-24.10' into impl-multi-…
b073b7d 
…process-stage
@yczhang-nv
add unit tests for shared_process_pool
f7b7280
@yczhang-nv
Fix SharedProcessPool unit test
40234e2
@yczhang-nv
fix multi_processing_stage unit test
d92ef9c
@yczhang-nv
add dataclass
26f8b0f
@yczhang-nv
add result_collection_loop
1c3666c
@yczhang-nv
add threading, need debug
5fca573
@yczhang-nv
Revise shared_process_pool design
8108fd4
@yczhang-nv
Add pool.status, need debug on stop elegantly
31e2247
@yczhang-nv
ut passed
6a7cf6e
@yczhang-nv
Improved and passed unit tests
fb2e877
@yczhang-nv
Merge remote-tracking branch 'upstream/branch-24.10' into impl-multi-…
be6eff1 
…process-stage
@yczhang-nv
Fix formats
dcda484
@yczhang-nv
fix format
16d46ba
@yczhang-nv
update multi_processing_stage
41de9df
@yczhang-nv
fix comments
5ae1fbd
@yczhang-nv
Fix pool status
546efdf
@yczhang-nv
fix test
506c0cf
@yczhang-nv
fix format
4c97e74
@yczhang-nv
Merge branch 'branch-24.10' of github.com:nv-morpheus/Morpheus into i…
a3e74f4 
…mpl-multi-process-stage
@yczhang-nv
fix CI
ee8633e
@yczhang-nv
initial commit for benchmarking
9749bbe
@yczhang-nv yczhang-nv self-assigned this Sep 20, 2024
@yczhang-nv yczhang-nv added the DO NOT MERGE PR should not be merged; see PR for details label Sep 20, 2024
@yczhang-nv yczhang-nv closed this Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@yczhang-nv yczhang-nv

Labels
DO NOT MERGE PR should not be merged; see PR for details
Projects
Morpheus Boards
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yczhang-nv