search

jina-ai / annlite

⚡ A fast embedded library for approximate nearest neighbor search
Apache License 2.0
216 stars 22 forks source link

feat: cibuildwheels #186

Closed OrangeSodahub closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #186 (f65dc22) into main (bb402ae) will not change coverage. The diff coverage is n/a.

:exclamation: Current head f65dc22 differs from pull request most recent head b572672. Consider uploading reports for the commit b572672 to get more accurate results

@@           Coverage Diff           @@
##             main     #186   +/-   ##
=======================================
  Coverage   81.65%   81.65%           
=======================================
  Files          20       20           
  Lines        1172     1172           
=======================================
  Hits          957      957           
  Misses        215      215
Flag Coverage Δ
annlite 81.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

OrangeSodahub commented 2 years ago

The function that uses hatch to build extensions has been under development. See https://github.com/pypa/hatch/issues/279. So give up hatch.

OrangeSodahub commented 2 years ago

For pip install -e ., annlite will be installed locally and import annlite will succeed in pytest, however pip install *.whl will install annlite to system env, import annlite will import annlite from local directions and have the problem of circular import. So change the pytest roodir from ./annlite to ./.

OrangeSodahub commented 2 years ago

See https://learn.microsoft.com/en-us/cpp/intrinsics/umulh?view=msvc-170, in fusefilter.h, the identifier __umulh requires (on windows):

Intrinsic Architecture
__umulh x64, ARM64