I don't know how often these features get used, but I would vote to not make this a big deal. I would say include it in the list of bugfixes in the next point release, but it may not necessitate an email to the whole list about the issue.

Notably, I ran into similar issue last week during some analysis with the cylindrical coordinate system, but I tested it first to see if it made sense. I anticipate that other users would do the same thing instead of just assuming its all correct, but maybe I'm wrong? I've been meaning to make a bug report on the cylindrical dataset, but maybe your PR has corrected the issues I encountered.

Thanks for your and Philipp's work to correct these issues!

Cameron

On Thu, Feb 8, 2018 at 12:09 PM, Nathan Goldbaum <nathan12343@gmail.com> wrote:

Hi all,

I'd like to point out a pull request I just made, which fixes a number of fields that have been returning incorrect data. In particular, the definitions of the fields `velocity_spherical_theta`, `velocity_spherical_phi`, and `velocity_cylindrical_theta` have not been correct for at least three years, possibly longer - I haven't checked to see if they were ever correct.

Here's a pull request that implements a fix and adds some new tests:

https://github.com/yt-project/yt/pull/1687

While it's nice that we know there's an issue now and that it's fixed, this means that if someone was making a profile of the tangential velocity around a star, or the poloidal magnetic field in a disk simulation, or a circular velocity in a galaxy simulation, yt may have silently returned incorrect results.

I'm raising this issue here because it feels like it might be a big deal. Should this concern be raised more loudly to our users? I'm hesitant to do so, mostly out of sheepishness. Is this as big a deal as I'm making it out to be?

Thanks,

Nathan

_______________________________________________
yt-dev mailing list -- yt-dev@python.org
To unsubscribe send an email to yt-dev-leave@python.org

_______________________________________________
yt-dev mailing list -- yt-dev@python.org
To unsubscribe send an email to yt-dev-leave@python.org

Cameron Hummels

NSF Postdoctoral Fellow

Department of Astronomy

California Institute of Technology

http://chummels.org