Pro-Enviro / IDNI

0 stars 0 forks source link

CO2e units incorrect #153

Closed adamshelley31 closed 3 weeks ago

adamshelley31 commented 2 months ago

Reported By James

Check bug report (Submitted 11:57, 4th September)

In Data > Recommendations > Breakdown of Energy Costs... In the table, total carbon emissions are 1/1000th of the actual sum. It is believed that this is due to a confusion with units. For example, the screenshot shows 10,210.76 tCO2e of carbon for electricity for the company cemcor, then the total is 10.21. The total should be 10,210.76 - no dividing needed.

mila-proenviro commented 3 weeks ago
Screenshot 2024-10-22 at 16 20 43

The numbers aren't showing or they are showing as zero

mila-proenviro commented 3 weeks ago

the issue is coming from getData() on generate-report image

mila-proenviro commented 3 weeks ago

Update: the issue is coming from calculateScopeTable - on line "const selectedConversionFactor" Currecntly, the conversion factor that we are pulling is from the conversionFactors object and that's why the HH is showing as 0 image

TO DO: the conversion factor needs to be the one from the object conversionFactors[fuelType.type] or the HH data conversion factor

mila-proenviro commented 3 weeks ago

TO DO:

  1. test different data
  2. check if the numbers are pulling through
  3. delete console logs and comments that we dont' need
mila-proenviro commented 3 weeks ago

Result

Screenshot 2024-10-23 at 12 19 25
mila-proenviro commented 3 weeks ago

If statement was added to handle custom fuel type names

Screenshot 2024-10-23 at 12 48 50