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

metadata fail the tests #935

Open
SaraZahoor opened this issue Mar 27, 2019 · 2 comments
Open

metadata fail the tests #935

SaraZahoor opened this issue Mar 27, 2019 · 2 comments

Comments

@SaraZahoor
Copy link

I have a rockstar catalog and when I try to retrieve the values of viral radius using columns_to_keep_dict, I get this:

The halo catalog and/or its associated metadata fail the following tests:

  1. All values of the halo_rvir column
    must be less than 50, crudely ensuring you used Mpc/h units.

Can anyone help here?

@aphearin
Copy link
Contributor

The problem is most likely that your halo_rvir column data is stored in kpc, rather than Mpc, so the data in this column need to have their units converted because Halotools assumes all distances in a halo catalog will be in the same units. This is a very common issue when processing Rockstar hlist files, and so there is a mechanism dedicated to resolving this difference in convention. Have a look at the docstring to the RockstarHlistReader.read_halocat method. The first argument is columns_to_convert_from_kpc_to_mpc, a list of strings. For any column you wish to read in and convert from kpc to Mpc, the name of the column should appear in this list.

@SaraZahoor
Copy link
Author

SaraZahoor commented Apr 1, 2019 via email

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

No branches or pull requests

2 participants