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

Mean bias #61

Merged
merged 56 commits into from Jan 13, 2023
Merged

Mean bias #61

merged 56 commits into from Jan 13, 2023

Conversation

henriquelettieri
Copy link
Contributor

The first pull request to implement the halo bias in master

@coveralls
Copy link

coveralls commented Oct 14, 2022

Coverage Status

Coverage: 49.032% (+3.03%) from 46.0% when pulling b9acf57 on mean_bias into 392bd34 on master.

Copy link
Contributor

@vitenti vitenti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a minimal documentation and unit tests to halo bias objects.

@vitenti
Copy link
Contributor

vitenti commented Dec 13, 2022

@henriquelettieri and @eduardojsbarroso this pull request is just waiting for you to add minimal documentation and unit tests to halo bias objects.

@eduardojsbarroso
Copy link
Contributor

Ok, I just don't know if I can do it this week

@vitenti vitenti added this to In progress in NumCosmo 1.0 via automation Jan 2, 2023
@eduardojsbarroso
Copy link
Contributor

eduardojsbarroso commented Jan 11, 2023

@henriquelettieri and @eduardojsbarroso this pull request is just waiting for you to add minimal documentation and unit tests to halo bias objects.

I didn't find a good reference for the bias in nc_halo_bias_st_ellip.c object. I can deduce the bias function from the code but I do not know what the new parameters mean to write on the documentation. Do you guys have a reference for this equation?

@vitenti vitenti merged commit ea3b7bb into master Jan 13, 2023
NumCosmo 1.0 automation moved this from In progress to Done Jan 13, 2023
@vitenti vitenti deleted the mean_bias branch January 13, 2023 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
NumCosmo 1.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants