search

UCBerkeleySETI / blimpy

Breakthrough Listen I/O Methods for Python
https://blimpy.readthedocs.io
BSD 3-Clause "New" or "Revised" License
46 stars 41 forks source link

Remove cython dependence #265

Closed jan-janssen closed 2 years ago

jan-janssen commented 2 years ago

Is cython still required?

codecov[bot] commented 2 years ago

Codecov Report

Merging #265 (d795e64) into master (18a8131) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #265   +/-   ##
=======================================
  Coverage   82.54%   82.54%           
=======================================
  Files          70       70           
  Lines        3695     3695           
=======================================
  Hits         3050     3050           
  Misses        645      645

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 18a8131...d795e64. Read the comment docs.

texadactyl commented 2 years ago

Yes, cython is not required. That entry in requirements.txt might have been a left-over from a few years ago.