Add temperature change card in site dashboard

[K-Markopoulos]

Issue #1022

Add 7-days temperature change card in place of Buoy Observation card when site doesn't have a spotter.

• Calculate the change based on satelliteTemperature from site's dailyData: avg of 7 last days minus avg of 8-14 days back
• Calculate the avg temp of the last 7 days.

[pullrequest]

⚠️ This pull request was not sent to the PullRequest network because the pull request is a draft.

[github-actions]

Build succeeded and deployed at https://aqualink-app-1046.surge.sh
(hash f356dfd deployed at 2024-10-31T10:15:20)

[ericboucher]

Note that we still want to display the Sensor card when there is a spotter/buoy with live data for that site

[Caesarh97]

Hi @K-Markopoulos,

I think many of the changes look great. However, I'd like to add some more information about the card. We would like the number in color to be calculated by the past 7 days compared to the 7 days before that. As Eric mentioned, this card should be shown if there's no buoy data available. If a site has water quality data and no buoy, we should still display the heat stress card together with the water quality card like before. I think we would want to show the temperature and one decimal number (ex. 22.6°C, just like you've done in the example).

I think this calculation would be the most accurate to calculate the past 7 days compared to the 7 days before that.
(sum of most recent 7 days) / 7 = (average temp for past 7 days)
(Sum of past 14-8 days / 7 = (average temp for past 14-8 days)
(average temp for past 7 days) - (average temp for past 14-8 days) = average 7-day change in temperature for the past 7 days compared to the previous 7 days.

Example:
Past 14 days (Day 14 - Day 1. Left to right)
23, 24, 26, 22, 19, 20, 24, 22, 25, 26, 25, 23, 23, 22
(Sum of 22, 25, 26, 25, 23, 23, 22) / 7 = 23.7 (average most recent 7 days)
(Sum of 23, 24, 26, 22, 19, 20, 24) / 7 = 22.6 (average past 14-8 days)
23.7-22.6=1.1
The average 7-day change in temperature for the past 7 days compared to the previous 7 days is 1.1°C.

[K-Markopoulos]

@Caesarh97 Thanks for the detailed explanation, I've updated the logic. Please let me know if anything seems off.

[pullrequest]

✅ This pull request was sent to the PullRequest network for review. Expert reviewers are now being matched to your request based on the code's requirements. Stay tuned!

What to expect from this code review:

• Comments posted to any areas of potential concern or improvement.
• Detailed feedback or actions needed to resolve issues that are found.
• Turnaround times vary, but we aim to be swift.

---

@K-Markopoulos you can click here to see the review status or cancel the code review job.

[pullrequest]

PullRequest Breakdown

Reviewable lines of change

+ 944
- 411

83% Jest Snapshot (tests)
13% TSX
4% TSX (tests)
1% TypeScript

Generated lines of change

+ 1
- 0

Type of change

Feature - These changes are adding a new feature or improvement to existing code.

[pullrequest]

I have no concerns with this pull request, security or otherwise.

Jacob

---

Reviewed with ❤️ by PullRequest

[pullrequest]

PullRequest reviewed the updates made to #1046 up until the latest commit (13fcbf8). No further issues were found.

Reviewed by:

Jacob

[pullrequest]

PullRequest reviewed the updates made to #1046 up until the latest commit (cfc9aa6). No further issues were found.

Reviewed by:

Jacob

[Caesarh97]

@K-Markopoulos That looks great! Thank you!