Andrew Hearin

I'm still not sure, but it looks like this might be related to the use of a lookup table for the concentration. For some values of the manually-overridden concentration, I...

If I change the default spacing in the concentration lookup table from dc=1 to dc=0.25, then the discrepancy gets quite a bit smaller, so I think this is the likely...

@EiffL here's my branch where I tried the more finely spaced concentrations - https://github.com/astropy/halotools/tree/mockpop_bugfix could you try running your tpcf test to see whether this improves the discrepancy you have...

Actually, you don't need to use a different branch. In the current `master` branch, you can test this hypothesis by passing in the `concentration_bins` argument to the `PrebuiltHodModelFactory`.

 

> so yeah, looks like this is solving the problem! Great, thanks for independently confirming that this is the issue. I'm glad this turned out to have a simple resolution....

Likewise, there should be a worked example of the `return_indexes` feature of CAM.

hmm, good point about the warning settings. I agree this is not good practice. Both me and @duncandc do this a lot, actually. It's probably easy to write a context...

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...

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