Ignore small wind direction changes with insignificant speed magnitude (#164)

* Ignore small wind direction changes if insignificant speed

* lceanup

Author: aeharding

src/features/rap/Row.tsx

@@ -0,0 +1,89 @@
+import styled from "@emotion/styled";
+import { WindsAloftAltitude, WindsAloftHour } from "../../models/WindsAloft";
+import Altitude from "./cells/Altitude";
+import Temperature from "./cells/Temperature";
+import WindDirection from "./cells/WindDirection";
+import WindSpeed from "./cells/WindSpeed";
+import { css } from "@emotion/react";
+import { vectorDifferenceMagnitude } from "../../helpers/vector";
+
+const DELTA_WINDSPEED_VECTOR_THRESHOLD_KPH = 10;
+
+const TableRow = styled.tr<{ opaque: boolean }>`
+  ${({ opaque }) =>
+    opaque &&
+    css`
+      opacity: 0.5;
+    `}
+`;
+
+interface RowProps {
+  datum: WindsAloftAltitude;
+  index: number;
+  displayedRapData: WindsAloftAltitude[];
+  surfaceLevel: number;
+  windsAloftHour: WindsAloftHour;
+}
+
+export default function Row({
+  datum,
+  index,
+  displayedRapData,
+  surfaceLevel,
+  windsAloftHour,
+}: RowProps) {
+  function negativeAltitude(datum: WindsAloftAltitude): boolean {
+    return !!(datum.altitudeInM - surfaceLevel < 0);
+  }
+
+  const shearEligible =
+    displayedRapData[index - 1] &&
+    vectorDifferenceMagnitude(
+      datum.windSpeedInKph,
+      datum.windDirectionInDeg,
+      displayedRapData[index - 1]?.windSpeedInKph,
+      displayedRapData[index - 1]?.windDirectionInDeg,
+    ) > DELTA_WINDSPEED_VECTOR_THRESHOLD_KPH;
+
+  return (
+    <TableRow key={index} opaque={negativeAltitude(datum)}>
+      <td>
+        <Altitude
+          heightInMeters={datum.altitudeInM}
+          surfaceLevelInMeters={surfaceLevel}
+          pressure={datum.pressure}
+        />
+      </td>
+      <td>
+        <Temperature
+          temperature={datum.temperatureInC}
+          dewpoint={datum.dewpointInC}
+          lapseRate={
+            displayedRapData[index - 1]
+              ? -(
+                  (datum.temperatureInC -
+                    displayedRapData[index - 1].temperatureInC) /
+                  (datum.altitudeInM - displayedRapData[index - 1].altitudeInM)
+                )
+              : undefined
+          }
+          pressure={datum.pressure}
+          hour={new Date(windsAloftHour.date)}
+        />
+      </td>
+      <td>
+        <WindDirection
+          curr={datum.windDirectionInDeg}
+          prev={displayedRapData[index - 1]?.windDirectionInDeg}
+          shearEligible={shearEligible}
+        />
+      </td>
+      <td>
+        <WindSpeed
+          curr={datum.windSpeedInKph}
+          prev={displayedRapData[index - 1]?.windSpeedInKph}
+        />
+      </td>
+    </TableRow>
+  );
+}

src/features/rap/Table.tsx

@@ -1,10 +1,5 @@
-import { css } from "@emotion/react";
 import styled from "@emotion/styled";
 import { useAppDispatch, useAppSelector } from "../../hooks";
-import Altitude from "./cells/Altitude";
-import Temperature from "./cells/Temperature";
-import WindDirection from "./cells/WindDirection";
-import WindSpeed from "./cells/WindSpeed";
 import { headerText } from "./CinCape";
 import { WindsAloftAltitude, WindsAloftHour } from "../../models/WindsAloft";
 import {
@@ -22,6 +17,7 @@ import { toggleAltitude } from "../user/userSlice";
 import { toggleAltitudeType } from "../../helpers/locale";
 import { notEmpty } from "../../helpers/array";
 import Tooltip from "../../shared/Tooltip";
+import Row from "./Row";
 
 const TableEl = styled.table`
   width: 100%;
@@ -35,14 +31,6 @@ const TableEl = styled.table`
   }
 `;
 
-const Row = styled.tr<{ opaque: boolean }>`
-  ${({ opaque }) =>
-    opaque &&
-    css`
-      opacity: 0.5;
-    `}
-`;
-
 const InteractTh = styled.th`
   cursor: pointer;
 `;
@@ -148,10 +136,6 @@ export default function Table({
     surfaceLevel,
   ]);
 
-  function negativeAltitude(datum: WindsAloftAltitude): boolean {
-    return !!(datum.altitudeInM - surfaceLevel < 0);
-  }
-
   return (
     <TableEl>
       <thead>
@@ -182,45 +166,14 @@ export default function Table({
 
       <tbody>
         {displayedRapData.map((datum, index) => (
-          <Row key={index} opaque={negativeAltitude(datum)}>
-            <td>
-              <Altitude
-                heightInMeters={datum.altitudeInM}
-                surfaceLevelInMeters={surfaceLevel}
-                pressure={datum.pressure}
-              />
-            </td>
-            <td>
-              <Temperature
-                temperature={datum.temperatureInC}
-                dewpoint={datum.dewpointInC}
-                lapseRate={
-                  displayedRapData[index - 1]
-                    ? -(
-                        (datum.temperatureInC -
-                          displayedRapData[index - 1].temperatureInC) /
-                        (datum.altitudeInM -
-                          displayedRapData[index - 1].altitudeInM)
-                      )
-                    : undefined
-                }
-                pressure={datum.pressure}
-                hour={new Date(windsAloftHour.date)}
-              />
-            </td>
-            <td>
-              <WindDirection
-                curr={datum.windDirectionInDeg}
-                prev={displayedRapData[index - 1]?.windDirectionInDeg}
-              />
-            </td>
-            <td>
-              <WindSpeed
-                curr={datum.windSpeedInKph}
-                prev={displayedRapData[index - 1]?.windSpeedInKph}
-              />
-            </td>
-          </Row>
+          <Row
+            key={index}
+            datum={datum}
+            index={index}
+            surfaceLevel={surfaceLevel}
+            windsAloftHour={windsAloftHour}
+            displayedRapData={displayedRapData}
+          />
         ))}
       </tbody>
     </TableEl>

src/features/rap/cells/WindDirection.tsx

@@ -40,21 +40,27 @@ const Container = styled.div<{ shear: boolean }>`
 interface WindDirectionProps {
   curr: number;
   prev?: number;
+  shearEligible: boolean;
 }
 
-export default function WindDirection({ curr, prev }: WindDirectionProps) {
+export default function WindDirection({
+  curr,
+  prev,
+  shearEligible,
+}: WindDirectionProps) {
   const content = useMemo(() => {
     return (
       <Container
         shear={
-          Math.abs(getAngleDifference(curr, prev === undefined ? curr : prev)) >
-          25
+          !!shearEligible &&
+          !!prev &&
+          Math.abs(getAngleDifference(curr, prev)) > 25
         }
       >
         {Math.round(curr)} <WindIndicator direction={curr} />
       </Container>
     );
-  }, [curr, prev]);
+  }, [curr, prev, shearEligible]);
 
   return content;
 }

src/helpers/vector.ts

@@ -0,0 +1,23 @@
+export function vectorDifferenceMagnitude(
+  speed1: number,
+  direction1: number,
+  speed2: number,
+  direction2: number,
+): number {
+  // Convert directions from degrees to radians
+  const radian1 = (Math.PI / 180) * direction1;
+  const radian2 = (Math.PI / 180) * direction2;
+
+  // Convert polar coordinates to Cartesian coordinates
+  const x1 = speed1 * Math.cos(radian1);
+  const y1 = speed1 * Math.sin(radian1);
+  const x2 = speed2 * Math.cos(radian2);
+  const y2 = speed2 * Math.sin(radian2);
+
+  // Calculate the difference in Cartesian coordinates
+  const dx = x2 - x1;
+  const dy = y2 - y1;
+
+  // Calculate the magnitude of the difference vector
+  return Math.sqrt(dx * dx + dy * dy);
+}