Hi Pengfei,<div><br></div><div>Can you <span></span>open an issue so we don't lose track of this?</div><div><br></div><div><a href="https://bitbucket.org/yt_analysis/yt/issues/new">https://bitbucket.org/yt_analysis/yt/issues/new</a><br></div><div><br></div><div>If you can, include a self-contained example using one of the cosmology time series datasets from <a href="http://yt-project.org/data">yt-project.org/data</a> so one of us can reproduce you issue locally.</div><div><br></div><div>Nathan</div><div><br>On Monday, August 8, 2016, Pengfei Chen <<a href="mailto:madcpf@gmail.com">madcpf@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>I was trying to generate light rays using yt 3.4-dev, but found that the light rays have wrong redshift intervals. For example, my simulation box is 80Mpccm/h per side, and I expect a light ray generated in a data dump at z=2 with length =1 simulation unit (i.e. 80Mpccm/h) would have a redshift interval from z=2 to z=1.92. However, the light ray generated by yt gives spans from z=2 to z=1.97.</div><div><br></div><div>As I look into this, I found that the function comoving_radial_distance in cosmology.py might return the wrong unit. I think the return value should be in comoving units, instead of physical units. To see this directly, I made the following change in cosmology_splice.py:</div><div><br></div><div><div>(root) ~/yt-conda/src/yt-hg/yt/<wbr>analysis_modules/cosmological_<wbr>observation $diff cosmology_splice.py cosmology_splice.py0</div><div>373d372<br></div><div><             print target_distance, distance2, z2</div></div><div><br></div><div>And it shows:</div><div><br></div><div><div>80.0 Mpccm/h 4.65320708035e+26 cm 1.97387969592 dimensionless</div><div>80.0 Mpccm/h 1.19455177073e+26 cm 1.97343368065 dimensionless</div><div>80.0 Mpccm/h 1.21507519065e+26 cm 1.97342592996 dimensionless</div></div><div><br></div><div><div>Then I made the following change in cosmology.py:</div><div><br></div><div><div>(root) ~/yt-conda/src/yt-hg/yt/<wbr>utilities $diff cosmology.py cosmology.py0</div><div>111,112c111,112</div><div><         return self.quan((self.hubble_<wbr>distance() *</div><div><                 trapzint(self.inverse_<wbr>expansion_factor, z_i, z_f)).value, 'cmcm')</div><div>---</div><div>>         return (self.hubble_distance() *</div><div>>                 trapzint(self.inverse_<wbr>expansion_factor, z_i, z_f)).in_base(self.unit_<wbr>system)</div></div></div><div><br></div><div><br></div><div>Then I get:</div><div><div>80.0 Mpccm/h 4.65320708035e+26 cmcm 1.92163908776 dimensionless</div><div>80.0 Mpccm/h 3.62771515661e+26 cmcm 1.92124702027 dimensionless</div></div><div><br></div><div>With the change of unit from cm to cmcm, the light rays have the right span.</div><div><br></div><div>Even though this solves my problem, I am not sure if similar problem still exists. For example, instead of making change in comoving_radial_distance, we might need to change hubble_distance into comoving units. Hopefully someone familiar with yt unit system could check this.</div><div><br></div><div>Thanks,<br>Pengfei</div>































</div>
</blockquote></div>