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

Move Pycoro from Morpheus to MRC #409

Merged
merged 10 commits into from
Oct 25, 2023

Conversation

cwharris
Copy link
Contributor

@cwharris cwharris commented Oct 24, 2023

Moves pycoro from Morpheus to MRC and incorperates tests from nv-morpheus/Morpheus#1286

Closes nv-morpheus/Morpheus#1268

@cwharris cwharris added improvement Improvement to existing functionality non-breaking Non-breaking change labels Oct 24, 2023
@cwharris cwharris requested review from a team as code owners October 24, 2023 20:19
@cwharris cwharris changed the title Move Pycoro tests from Morpheus to MRC Move Pycoro from Morpheus to MRC Oct 24, 2023
Copy link
Contributor

@drobison00 drobison00 left a comment

Choose a reason for hiding this comment

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

Looks good overall; probably need to figure out some way to test the fiber based paths at some point, it doesn't look like anything hits them at the moment.

python/tests/test_pycoro.py Outdated Show resolved Hide resolved
@cwharris
Copy link
Contributor Author

@drobison00 yeah the only way to do that is with the CoroutineRunnable. That's a separate issue and the next PR I'm working on.

@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #409 (5412e76) into branch-23.11 (36460c9) will increase coverage by 0.16%.
The diff coverage is 89.33%.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff                @@
##           branch-23.11     #409      +/-   ##
================================================
+ Coverage         73.27%   73.44%   +0.16%     
================================================
  Files               389      392       +3     
  Lines             13779    13862      +83     
  Branches           1043     1050       +7     
================================================
+ Hits              10097    10181      +84     
+ Misses             3682     3681       -1     
Flag Coverage Δ
cpp 69.31% <0.00%> (-0.09%) ⬇️
py 42.59% <89.33%> (+1.02%) ⬆️

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

Files Coverage Δ
python/mrc/_pymrc/src/coro.cpp 100.00% <100.00%> (ø)
python/mrc/core/coro.cpp 100.00% <100.00%> (ø)
python/mrc/_pymrc/include/pymrc/coro.hpp 87.30% <87.30%> (ø)

... and 5 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@cwharris
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 1ebd4e2 into nv-morpheus:branch-23.11 Oct 25, 2023
14 checks passed
rapids-bot bot pushed a commit to nv-morpheus/Morpheus that referenced this pull request Oct 27, 2023
Depends on nv-morpheus/MRC#409

Removes the the pycoro functionality in morpheus in favor of the same functionality which was moved to mrc.

Authors:
  - Christopher Harris (https://github.com/cwharris)

Approvers:
  - Devin Robison (https://github.com/drobison00)

URL: #1313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to existing functionality non-breaking Non-breaking change
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

[IMP]: Add tests for pycoro binding code
2 participants