search

kevin218 / Eureka

Eureka! is a data reduction and analysis pipeline intended for time-series observations with JWST.
https://eurekadocs.readthedocs.io/
MIT License
60 stars 47 forks source link

Removing Cython as a dependency #578

Closed taylorbell57 closed 11 months ago

taylorbell57 commented 11 months ago

Cython was only needed for niriss_cython.pyx which (since we currently don't support NIRISS) could just be a python file instead of a cython file. This will hopefully resolve our GitHub automated testing (we'll see with this PR's tests).

codecov-commenter commented 11 months ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev/kbs@3ee2cf6). Click here to learn what that means. The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             dev/kbs     #578   +/-   ##
==========================================
  Coverage           ?   57.59%           
==========================================
  Files              ?       96           
  Lines              ?    11518           
  Branches           ?        0           
==========================================
  Hits               ?     6634           
  Misses             ?     4884           
  Partials           ?        0

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

taylorbell57 commented 11 months ago

Oops, forgot to remove cython from setup.cfg. I'll make another commit soon