[yt-svn] commit/yt: xarthisius: Merged in ngoldbaum/yt (pull request #2275)

[commits-noreply at bitbucket.org]

1 new commit in yt:

https://bitbucket.org/yt_analysis/yt/commits/d893be12e55c/
Changeset:   d893be12e55c
Branch:      yt
User:        xarthisius
Date:        2016-07-16 15:59:01+00:00
Summary:     Merged in ngoldbaum/yt (pull request #2275)

Add minus sign to definition of sz_kinetic field. Closes #1225
Affected #:  1 file

diff -r baec4945cd28a161c1b7b42c63840c9626da7174 -r d893be12e55c80f5fce09fe679cd570b0b7cc1f3 yt/fields/astro_fields.py
--- a/yt/fields/astro_fields.py
+++ b/yt/fields/astro_fields.py
@@ -131,7 +131,9 @@
         if vel_axis > 2:
             raise NeedsParameter(["axis"])
         vel = data[ftype, "velocity_%s" % ({0: "x", 1: "y", 2: "z"}[vel_axis])]
-        return scale * vel * data[ftype, "density"]
+        # minus sign is because radial velocity is WRT viewer
+        # See issue #1225
+        return -scale * vel * data[ftype, "density"]
 
     registry.add_field((ftype, "sz_kinetic"),
                        function=_sz_kinetic,

Repository URL: https://bitbucket.org/yt_analysis/yt/

--

This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.