Merge pull request #1628 from ORNL-AMO/issue-1624

fix rounding issue in reports and summation in excel BP report
ORNL-AMO · Jun 17, 2024 · 3c5b219 · 3c5b219
2 parents 713d557 + b58386a
commit 3c5b219
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/src/app/shared/helper-pipes/custom-number.pipe.ts b/src/app/shared/helper-pipes/custom-number.pipe.ts
@@ -9,7 +9,7 @@ export class CustomNumberPipe implements PipeTransform {
   transform(value: number, isCurrency?: boolean): string {
     let valueStr: string;
     if (isNaN(value) == false && value != null) {
-      if (value < 10000) {
+      if (Math.abs(value) < 10000) {
         //5 sig figs
         valueStr = (value).toLocaleString(undefined, { maximumSignificantDigits: 5 });
       } else {

diff --git a/src/assets/csv_templates/BBBP-Challenge-Annual-Reporting-Form.xlsx b/src/assets/csv_templates/BBBP-Challenge-Annual-Reporting-Form.xlsx