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.

Sign up for GitHub

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

Jump to bottom

model.mock.estimate_ngals() doesn't work for composite HOD model #801

Closed
KuanWang-Astro opened this issue Aug 28, 2017 · 1 comment
Closed

model.mock.estimate_ngals() doesn't work for composite HOD model #801

KuanWang-Astro opened this issue Aug 28, 2017 · 1 comment
Assignees
@aphearin
Labels
bug empirical-models
Milestone
v0.6

Comments

@KuanWang-Astro
Copy link

image
When I populate the halo catalog with this model, the estimate_ngals() method gives back an error:
image
while the following works.
image
The only difference should be the HOD model.
@aphearin aphearin self-assigned this Aug 28, 2017
@aphearin aphearin added this to the v0.6 milestone Aug 28, 2017
@aphearin
Copy link
Contributor

Thanks for pointing this out @KuanWang-Astro - should be an easy fix.

aphearin pushed a commit to aphearin/halotools that referenced this issue Nov 27, 2017
Resolved estimate_ngals bug pointed out by @KuanWang-Astro in Issue a…
5d5dadd 
…stropy#801.
@aphearin aphearin mentioned this issue Nov 27, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@aphearin aphearin

Labels
bug empirical-models
Projects
None yet
Milestone
v0.6
Development

No branches or pull requests
2 participants
@aphearin @KuanWang-Astro