Fix for [#31813] "bge.types.KX_RadarSensor incorrect attributes" reported by Monster.
authorMitchell Stokes <mogurijin@gmail.com>
Thu, 14 Jun 2012 08:01:20 +0000 (08:01 +0000)
committerMitchell Stokes <mogurijin@gmail.com>
Thu, 14 Jun 2012 08:01:20 +0000 (08:01 +0000)
KX_RadarSensor.angle was returning the angle that was used to construct Bullet's physics shape, which is calculated from the logic brick gui. KX_RadarSensor.angle now recalculates the original value from the gui. However, m_coneradius isn't actually used by KX_RadarSensor that I can see, so it might be better to just assign the original angle to m_coneradius instead of the calculated value. I've also made KX_RadarSensor.angle read-only, since setting m_coneradius does not appear to have any affect, which means writing to KX_RadarSensor.angle never worked properly.

doc/python_api/rst/bge.types.rst
source/gameengine/Ketsji/KX_RadarSensor.cpp
source/gameengine/Ketsji/KX_RadarSensor.h

index bcef816dc9d2a577b188ad7cefc4290f701f8f1e..1d9f84fbcac6c01308c6b14f23b4d9a6d3cc8dea 100644 (file)
@@ -2692,7 +2692,7 @@ Game Types (bge.types)
 
       The angle of the cone (in degrees) with which to test.
 
-      :type: float from 0 to 360
+      :type: float
 
    .. attribute:: axis
 
index 678794f2bebaaf151fd498db680a8fc36b59aaeb..978944c20e866063518dc9bed8b321a2f917577f 100644 (file)
@@ -212,9 +212,19 @@ PyAttributeDef KX_RadarSensor::Attributes[] = {
        KX_PYATTRIBUTE_FLOAT_ARRAY_RO("coneOrigin", KX_RadarSensor, m_cone_origin, 3),
        KX_PYATTRIBUTE_FLOAT_ARRAY_RO("coneTarget", KX_RadarSensor, m_cone_target, 3),
        KX_PYATTRIBUTE_FLOAT_RO("distance", KX_RadarSensor, m_coneheight),
-       KX_PYATTRIBUTE_FLOAT_RW("angle", 0, 360, KX_RadarSensor, m_coneradius),
+       KX_PYATTRIBUTE_RO_FUNCTION("angle", KX_RadarSensor, pyattr_get_angle),
        KX_PYATTRIBUTE_INT_RW("axis", 0, 5, true, KX_RadarSensor, m_axis),
        {NULL} //Sentinel
 };
 
+PyObject* KX_RadarSensor::pyattr_get_angle(void *self_v, const KX_PYATTRIBUTE_DEF *attrdef)
+{
+       KX_RadarSensor* self= static_cast<KX_RadarSensor*>(self_v);
+
+       // The original angle from the gui was converted, so we recalculate the value here to maintain
+       // consistency between Python and the gui
+       return PyFloat_FromDouble(MT_degrees(atan(self->m_coneradius / self->m_coneheight)) * 2);
+       
+}
+
 #endif // WITH_PYTHON
index 903413ca89bcf49aabaa5bc50e35b990b5137f4d..f1ed5b3c98237aaa64b8614d77acbe2de7a0b049 100644 (file)
@@ -93,6 +93,7 @@ public:
        /* python */
        virtual sensortype GetSensorType() { return ST_RADAR; }
 
+       static PyObject*        pyattr_get_angle(void *self_v, const KX_PYATTRIBUTE_DEF *attrdef);
 };
 
 #endif //__KX_RADARSENSOR_H__