-
Notifications
You must be signed in to change notification settings - Fork 14
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
[Issue]: Tests - Python API: Tests failures & Optimization #220
Comments
@SundarRajan98 @swetha097 please look at this issue |
@SundarRajan28 @swetha097 fails because tries to get dataset from source https://github.com/ROCm/rocAL/blob/cc619f581abb2b40d5b18cdae2550b79ba757e00/tests/python_api/decoder.py#L11C1-L11C11 |
@kiritigowda @LakshmiKumar23
|
@SundarRajan28 - with mainline build 14935+ you will be able to test rocAL PyBind with the package install. Below is the steps to test with docker
Output
|
@kiritigowda The steps shared above doesn't work in pytorch dockers where there are multiple python versions and pytorch is installed in the conda python. The pybind tests works fine when they are run normally but when ctests are built using the CMake, it detects the system python version and thus it fails due to module missing error. With python 3.10
With python 3.12
With ctests
|
@kiritigowda I've confirmed that the pybind ctests are working by manually changing the python version in the pybind tests CMake. If a flag is added like the PYTHON_VERSION_SUGGESTED, the user can pass the python version to run the ctests with.
ctests results
|
#224 fixes decoder.py issue created by @kiritigowda |
@SundarRajan28 - can you make the suggested change for multi-version support. Thanks! |
@SundarRajan28 - please verify if all the test scripts from Python API can be run with TOT. |
Problem Description
Operating System
ALL
CPU
ALL
GPU
AMD Instinct MI300X
ROCm Version
ROCm 6.2.0
ROCm Component
No response
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response
The text was updated successfully, but these errors were encountered: