From 3457576b4ef91c6995f7c366f379da101673e5bf Mon Sep 17 00:00:00 2001 From: Piotr Jankowski Date: Fri, 7 Jan 2022 14:58:21 +0100 Subject: [PATCH] Fix reduction_ambition to correctly use s2 Use s2.coverage_s2 instead of s2.coverage_s1 and s2.base_year_ghg_s2 instead of s2.base_year_ghg_s1 This should cause the combined s1+s2 to correctly account for the s2 part in target.reduction_ambition calculation --- SBTi/target_validation.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/SBTi/target_validation.py b/SBTi/target_validation.py index 6a6eb11..fe5ab33 100644 --- a/SBTi/target_validation.py +++ b/SBTi/target_validation.py @@ -168,9 +168,9 @@ def _combine_s1_s2(self, target: IDataProviderTarget): target.reduction_ambition * target.coverage_s1 * target.base_year_ghg_s1 - + s2.reduction_ambition * s2.coverage_s1 * s2.base_year_ghg_s2 + + s2.reduction_ambition * s2.coverage_s2 * s2.base_year_ghg_s2 ) - / (target.base_year_ghg_s1 + s2.base_year_ghg_s1) + / (target.base_year_ghg_s1 + s2.base_year_ghg_s2) / combined_coverage ) target.coverage_s1 = combined_coverage