search

simonsobs / pixell

A rectangular pixel map manipulation and harmonic analysis library derived from Sigurd Naess' enlib.
Other
42 stars 32 forks source link

remove auto-install for the users #244

Closed ickc closed 8 months ago

ickc commented 11 months ago

This patch remove logic that auto install cython for the users. I think it is a bit intrusive to the users and the users should be informed to change their environment manually if needed.

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d32c6a2) 43.72% compared to head (08fec03) 43.72%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #244 +/- ## ======================================= Coverage 43.72% 43.72% ======================================= Files 34 34 Lines 10940 10940 ======================================= Hits 4783 4783 Misses 6157 6157 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

msyriac commented 9 months ago

Agreed. Did you mean to leave this as a draft or is it ready to merge?

ickc commented 9 months ago

I was trying to wait for the test to pass first but then I was on annual leave. It's ready now.