medspacy icon indicating copy to clipboard operation
medspacy copied to clipboard

fixed issue where quickumls was ignored in medspacy load method

Open turbosheep opened this issue 2 years ago • 2 comments

Made it so quickumls is not ignored on medspacy.load now that install for quickumls has been improved on windows

turbosheep avatar Jun 16 '23 17:06 turbosheep

Failed to install unqlite Building wheels for collected packages: unqlite Building wheel for unqlite (setup.py): started Building wheel for unqlite (setup.py): finished with status 'error' error: subprocess-exited-with-error

× python setup.py bdist_wheel did not run successfully. │ exit code: 1 ╰─> [44 lines of output] /private/var/folders/mm/pltwc2yj1jx192t6dzy9zsrr0000gn/T/pip-install-fqp50fxn/unqlite_8ee989ba66064dcc998c69dcc0ed6824/setup.py:11: UserWarning: Cython not installed, using pre-generated C source file. warnings.warn('Cython not installed, using pre-generated C source file.') /Users/runner/hostedtoolcache/Python/3.9.18/x64/lib/python3.9/site-packages/setuptools/init.py:80: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated. !!

          ********************************************************************************
          Requirements should be satisfied by a PEP 517 installer.
          If you are using pip, you can try `pip install --use-pep517`.
          ********************************************************************************

Don't recall how we fixed in current dev version. It might be easier to close this and start a new pull from the latest dev instead? Thoughts? @turbosheep ?

jianlins avatar Jan 19 '24 23:01 jianlins

I see this in the log. Have we tried making sure that Cython is installed to prevent this?

warnings.warn('Cython not installed, using pre-generated C source file.')

burgersmoke avatar Jan 20 '24 02:01 burgersmoke