Closed danrademacher closed 2 years ago
Do they want to drop the labels ("Building Type Average", "WASH MUTUAL TOWER (1201 THIRD AVE)" shown on the far right here?
They are not in the design, but that could just be an oversight
@tsinn can you weigh in on those labels?
Sorry - I think it's helpful to include just like you've done, Tom. Looks good!
I think they are calculating percent change wrong here:
const previousValue = yearData[0].val;
const selectedValue = yearData[1].val;
return ((selectedValue - previousValue) / selectedValue) * 100;
They are dividing the difference by the current value, not the initial value
The formula should be:
( current_value - initial_value ) / initial_value * 100
Simple real-life example from last comment
(3-2)/3 = 33%
vs
(3-2)/2=50%
New design:![image](https://user-images.githubusercontent.com/1423200/144135349-240e245a-eb9a-43a2-8055-8b43b8171a75.png)
Currently looks like:![image](https://user-images.githubusercontent.com/1423200/144135375-252ce264-5501-4676-9828-6228f16da8b8.png)
Minimal changes:
┆Issue is synchronized with this Asana task