YuliangXiu / ECON

[CVPR'23, Highlight] ECON: Explicit Clothed humans Optimized via Normal integration
https://xiuyuliang.cn/econ
Other
1.08k stars 106 forks source link

CoLab Failure / Crash Step 10 #43

Closed jnevin closed 1 year ago

jnevin commented 1 year ago

Hey YuliangXiu

This new issue just raised on Step 10 of ECON Colab:

image

jnevin commented 1 year ago

@YuliangXiu - i repeated prior Step 9 just to make sure, and still had above failure in Step 10. Looks like an issue with the ext_modules option settings

kwan3854 commented 1 year ago

I experienced the same error message as you did. I found out that the issue started occurring after yesterday's GitHub commit. You can resolve the problem by using the previous commit version. Modify the content of the 5th cell in Colab as follows:

# Clone the repository
!git clone https://github.com/YuliangXiu/ECON/
%cd ECON
!git checkout f93dcd94ff06abcc71a64379d9d26dd1642f495f
YuliangXiu commented 1 year ago

Fixed.

This code generates the error:

from Cython.Build import cythonize
from setuptools import find_packages, setup

This code does not generate the error:

from setuptools import find_packages, setup
from Cython.Build import cythonize

[1] https://stackoverflow.com/questions/21594925/error-each-element-of-ext-modules-option-must-be-an-extension-instance-or-2-t

jnevin commented 1 year ago

@YuliangXiu wow - good catch! thanks I will re-test.