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

Support python coverage test. #649

Closed

Conversation

robinwnv
Copy link
Contributor

If we specify --cov_bin, it will run legate command with "coverage run". coverage here is a python module which measures python source code coverage.
e.g.:
./test.py --use=cuda --cov_bin /home/robinw/miniconda3/envs/legate/bin/coverage --cov_src_path /home/robinw/legate/cunumeric/cunumeric --debug --verbose

If we don't specify --cov_bin, it will run legate command without "coverage run" as before.
./test.py --use=cuda --debug --verbose

@robinwnv robinwnv added the category:task PR is a simple task and will not be included in release notes label Oct 10, 2022
@robinwnv robinwnv requested a review from bryevdv October 10, 2022 09:53
@bryevdv
Copy link
Contributor

bryevdv commented Oct 10, 2022

@robinw0928 You will need to close this and re-open against legate.core. All the test code moved there in nv-legate/legate.core#397 I was going to merge the cunumeric side of that work #627 late last week but some build issues came up. I will try to get it merged today.

+ [str(test_path)]
)
else:
cmd = [str(config.legate_path), str(test_path)]
Copy link
Contributor

@bryevdv bryevdv Oct 10, 2022

Choose a reason for hiding this comment

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

Also when you do move this to the legate repo, please change this to leave cmd the way it is. Construct the cov args in a method self.cov_args, similar to stage, file args, etc. and then add them to cmd in the line just below in the same way. Then there can be a test added to verify that cov_args are being constructed correctly.

@bryevdv
Copy link
Contributor

bryevdv commented Oct 10, 2022

@robinw0928 #627 has been merged

@bryevdv
Copy link
Contributor

bryevdv commented Oct 12, 2022

@robinw0928 I see you have submitted nv-legate/legate.core#431 I will go ahead and close this PR

@bryevdv bryevdv closed this Oct 12, 2022
@robinwnv robinwnv deleted the enable_coverage_test branch March 14, 2023 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:task PR is a simple task and will not be included in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants