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

ra_dec_z() improvement/bug #933

Open
duncandc opened this issue Feb 21, 2019 · 2 comments · May be fixed by #1008
Open

ra_dec_z() improvement/bug #933

duncandc opened this issue Feb 21, 2019 · 2 comments · May be fixed by #1008
Assignees
@duncandc
Labels
bug mock-observables

Comments

@duncandc
Copy link
Contributor

The default cosmology for the ra_dec_z() function has an error. H0=0.7 instead of 70. I addition, the redshift range it works for is only between 0-1. Both of these should be fixed.
@duncandc duncandc self-assigned this Feb 21, 2019
@aphearin
Copy link
Contributor

Hmmm, are you sure we should continue to support ra-dec-z functionality? Halotools turned out to not really be a lightcone code...

@duncandc
Copy link
Contributor Author

I have no problem with deleting it, but multiple students at Pitt have used it. This might be better as code just sitting in its own repository.

@pllim pllim linked a pull request Sep 9, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duncandc duncandc

Labels
bug mock-observables
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

WIP PR resolving #993 aphearin/halotools
2 participants
@duncandc @aphearin