warp-drive icon indicating copy to clipboard operation
warp-drive copied to clipboard

Fixing pycuda unit tests after example script migration

Open UsaidPro opened this issue 3 months ago • 3 comments

Hello! I had a test failure running the python warp_drive/utils/unittests/run_trainer_tests.py unit test in the README from the below error:

ImportError while importing test module '/home/usaidm/Documents/ExternalLibs/warp-drive/tests/wd_training/pycuda_tests/test_env_training.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
../../../.pyenv/versions/3.8.10/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/wd_training/pycuda_tests/test_env_training.py:14: in <module>
    from warp_drive.training.example_training_script_pycuda import setup_trainer_and_train
E   ModuleNotFoundError: No module named 'warp_drive.training.example_training_script_pycuda'

Decided to fix and submit a pull request since it is a small fix. Hope it helps!

After making this change, the unit test succeeds.

UsaidPro avatar Mar 26 '24 02:03 UsaidPro

Thanks for the contribution! Before we can merge this, we need @UsaidPro to sign the Salesforce Inc. Contributor License Agreement.

salesforce-cla[bot] avatar Mar 26 '24 02:03 salesforce-cla[bot]

Tried signing the CLA but got this failure: Could not sign the CLA, please contact: [email protected]. :(

EDIT: Clicking on the CLA link again results in a page stating "You have already signed the CLA Agreement". Is it possible to rerun the check?

UsaidPro avatar Mar 26 '24 02:03 UsaidPro

@UsaidPro Thank you for fixing this. I have approved your PR. Regarding this CLA, could you please refresh and check if it works? I am not sure why it is not working on your side, if it is still not working, please let me know, I can ask our open source team to find the reason.

Emerald01 avatar Mar 26 '24 16:03 Emerald01