simonsobs / pspy

Python power spectrum code
Other
7 stars 5 forks source link

replace .dot by @, since xavier found its faster for large matrices #49

Closed thibautlouis closed 1 year ago

codecov-commenter commented 1 year ago

Codecov Report

Merging #49 (2827366) into master (eeb21c8) will increase coverage by 0.27%. The diff coverage is 0.00%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/simonsobs/pspy/pull/49/graphs/tree.svg?width=650&height=150&src=pr&token=8HQ0DZDROW&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=simonsobs)](https://app.codecov.io/gh/simonsobs/pspy/pull/49?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=simonsobs) ```diff @@ Coverage Diff @@ ## master #49 +/- ## ========================================== + Coverage 28.18% 28.45% +0.27% ========================================== Files 18 18 Lines 1948 1954 +6 ========================================== + Hits 549 556 +7 + Misses 1399 1398 -1 ``` | [Impacted Files](https://app.codecov.io/gh/simonsobs/pspy/pull/49?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=simonsobs) | Coverage Δ | | |---|---|---| | [pspy/so\_cov.py](https://app.codecov.io/gh/simonsobs/pspy/pull/49?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=simonsobs#diff-cHNweS9zb19jb3YucHk=) | `7.97% <0.00%> (+0.04%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/simonsobs/pspy/pull/49/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=simonsobs)
xgarrido commented 1 year ago

LGTM