Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Individual contributions to the freq-freq cross #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dpole
Copy link
Contributor

@dpole dpole commented Feb 21, 2020

It applies only to CorrelatedFactorizedCrossSpectrum

It applies only to CorrelatedFactorizedCrossSpectrum
@codecov-io
Copy link

Codecov Report

Merging #31 into master will decrease coverage by 0.31%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   64.01%   63.69%   -0.32%     
==========================================
  Files           4        4              
  Lines         289      292       +3     
==========================================
+ Hits          185      186       +1     
- Misses        104      106       +2
Impacted Files Coverage Δ
fgspectra/cross.py 53.94% <33.33%> (-0.85%) ⬇️

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 161a61b...4b84ada. Read the comment docs.

@dpole dpole mentioned this pull request Feb 21, 2020
@dpole
Copy link
Contributor Author

dpole commented Nov 6, 2020

@xzackli and @mabitbol , I don't even remember who requested this feature, but I guess we can either merge or delete this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants