<div dir="ltr">I think the original intention was that obtain_rvec was for getting the radius vector and obtain_rv_vec was for getting the radial velocity vector. Looking at the code, I think these functions have been repurposed for alternate purposes in the intervening years.<div><br></div><div>Having one function for getting data like this sounds like a good idea, but care must be taken to make sure that the implementations really are redundant.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 9:55 AM, Britton Smith <span dir="ltr"><<a href="mailto:brittonsmith@gmail.com" target="_blank">brittonsmith@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Matt,<div><br></div><div>Yes, we can do that.  They should definitely be removed with extreme care, like the funny bone in Operation!, but perhaps it's a good idea to let the test suite run over it in a PR.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Britton</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 27, 2015 at 2:41 PM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Britton,<br>
<br>
Sounds like a mess to me.  I'm epsilon nervous about changing them,<br>
since we've had lots of turnover in this stuff, but do you want to go<br>
over these in a quick hangout collaboratively?<br>
<div><div><br>
On Tue, Oct 27, 2015 at 9:39 AM, Britton Smith <<a href="mailto:brittonsmith@gmail.com" target="_blank">brittonsmith@gmail.com</a>> wrote:<br>
> Hi all,<br>
><br>
> It seems that there are two definitions in the yt source of the following<br>
> functions:<br>
> - obtain_rvec<br>
> - obtain_rv_vec<br>
><br>
> For each of these, one is in utilities/lib/misc_utilities.pyx and the other<br>
> is in utilities/lib/geometry_utils.pyx.  Also in each case, the source code<br>
> for each is extremely similar, with the versions in misc_utilities<br>
> containing additional keywords.  At least in the case of obtain_rv_vec, both<br>
> versions are called in different places in the code.  Does anyone know of a<br>
> reason for this?  If there is one, this looks like a mess that should<br>
> probably be cleaned up.<br>
><br>
> Britton<br>
><br>
</div></div>> _______________________________________________<br>
> yt-dev mailing list<br>
> <a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
><br>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" rel="noreferrer" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br></div>