microsoft / hummingbird

Hummingbird compiles trained ML models into tensor computation for faster inference.
MIT License
3.32k stars 274 forks source link

TVM MacOS pipeline failing again #722

Closed ksaur closed 12 months ago

ksaur commented 12 months ago

In Build TVM if Mac step, see run. Not transient/never passes.

...
[100%] Built target tvm
Requirement already satisfied: wheel in /Users/runner/hostedtoolcache/Python/3.10.12/x64/lib/python3.10/site-packages (0.40.0)
Requirement already satisfied: packaging in /Users/runner/hostedtoolcache/Python/3.10.12/x64/lib/python3.10/site-packages (23.1)
python: can't open file '/Users/runner/work/hummingbird/tvm/build/setup.py': [Errno 2] No such file or directory
Error: Process completed with exit code 2.

Looking to see if some path changed on TVM end. The pipeline is expecting setup at tvm/build/setup.py...

cc @mshr-h

ksaur commented 12 months ago

From attaching to pipeline:

bash-3.2$ ls /Users/runner/work/hummingbird/tvm/build
CMakeCache.txt          TVMBuildOptions.txt     libtvm_runtime.dylib
CMakeFiles              cmake_install.cmake     temp_config_file.cmake
Makefile                compile_commands.json   tvmConfig.cmake

bash-3.2$ ls /Users/runner/work/hummingbird/tvm/python
gen_requirements.py     setup.py                tvm

I think the problem is with calling the setup.py from the ../tvm/build dir. Maybe I didn't understand #718 which moved the files. (I think when that was passing it was because it was hitting the cache? But I am not sure, it's possible I'm confused on something!)

    - name: Build TVM if Mac
      if: ${{ steps.cache.outputs.cache-hit != 'true' && startsWith(matrix.os, 'macos') }}
      working-directory: ../tvm/build
      run: |
        make -j3
        python -m pip install -U wheel packaging
        python setup.py bdist_wheel

In terms of the wheel packaging step itself, there were a lot of recent changes. Now we have some issues with building the wheel...but i'll try to fix those next