ENH: Resolve incorrect aggregation warning (#96)

* FIX: json NaN representation in assets/for_tests * FIX: update unit test assets to use NaNs where missing instead of 0 * ENH: update how to handle missing merge data * FIX: correct logger in cli * MAINT: ignore dunder main in coverage
loganthomas · Nov 10, 2022 · fcbdba6 · fcbdba6
1 parent 3d701f8
commit fcbdba6
Show file tree

Hide file tree

Showing 8 changed files with 90 additions and 82 deletions.
diff --git a/.coveragerc b/.coveragerc
@@ -1,3 +1,7 @@
 [run]
 omit =
     setup.py
+
+[report]
+exclude_lines =
+    if __name__ == .__main__.:
diff --git a/assets/for_tests/mock_leader_board.json b/assets/for_tests/mock_leader_board.json
@@ -13,7 +13,7 @@
     "margin": {
       "0": 6.61999999999999,
       "1": 6.579999999999998,
-      "2": NaN
+      "2": null
     },
     "pts_back": {
       "0": 0.0,
@@ -192,7 +192,7 @@
       "6": 1.0,
       "7": 7.0,
       "8": -3.0,
-      "9": 0.0
+      "9": null
     },
     "PROJ_pts": {
       "0": 17.9,
@@ -204,7 +204,7 @@
       "6": 1.02,
       "7": 6.57,
       "8": 6.6,
-      "9": 0.0
+      "9": null
     }
   }
 }
diff --git a/assets/for_tests/mock_leader_board_no_actual_pts.json b/assets/for_tests/mock_leader_board_no_actual_pts.json
@@ -13,7 +13,7 @@
     "margin": {
       "0": 0.0,
       "1": 0.0,
-      "2": NaN
+      "2": null
     },
     "pts_back": {
       "0": 0,
@@ -192,7 +192,7 @@
       "6": 0,
       "7": 0,
       "8": 0,
-      "9": 0
+      "9": null
     },
     "PROJ_pts": {
       "0": 17.9,
@@ -204,7 +204,7 @@
       "6": 1.02,
       "7": 6.57,
       "8": 6.6,
-      "9": 0.0
+      "9": null
     }
   }
 }