Skip to content

[skip ci][CI] Bump Python version from 3.6 to 3.7 in VitisAI.cmake - #10656

Merged
Mousius merged 1 commit into
apache:mainfrom
leandron:vitis-python37-fix
Mar 17, 2022
Merged

[skip ci][CI] Bump Python version from 3.6 to 3.7 in VitisAI.cmake#10656
Mousius merged 1 commit into
apache:mainfrom
leandron:vitis-python37-fix

Conversation

@leandron

Copy link
Copy Markdown
Contributor

This is required because we migrated the default version in which dependencies are installed, due to Python 3.6 coming to EOL.

Otherwise, when running on machines updated to use Python 3.7 by default, you'll see:


-- Found PythonInterp: /usr/bin/python3.6 (found suitable version "3.6.9", minimum required is "3.6") Traceback (most recent call last):
File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'pyxir'
CMake Error at cmake/modules/contrib/VitisAI.cmake:36 (message):
Can't build TVM with Vitis-AI because PyXIR can't be found
Call Stack (most recent call first):
CMakeLists.txt:480 (include)

It needs to be merged after #10654.

cc @jtuyls@areusch@driazati

@github-actions
github-actionsBot requested a review from areuschMarch 17, 2022 13:45
@leandronleandron changed the title Bump Python version from 3.6 to 3.7 in VitisAI.cmake[CI] Bump Python version from 3.6 to 3.7 in VitisAI.cmakeMar 17, 2022
This is required because we migrated the default version in which
dependencies are installed, due to Python 3.6 coming to EOL.
Co-authored-by: Elen Kalda <Elen.Kalda@arm.com>
@leandronleandron changed the title [CI] Bump Python version from 3.6 to 3.7 in VitisAI.cmake[skip ci][CI] Bump Python version from 3.6 to 3.7 in VitisAI.cmakeMar 17, 2022
@Mousius
Mousius merged commit 77c03d1 into apache:mainMar 17, 2022
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
This is required because we migrated the default version in which
dependencies are installed, due to Python 3.6 coming to EOL.
Co-authored-by: Elen Kalda <Elen.Kalda@arm.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@leandron@Mousius