try fix citi bike hello world dashboard (#555)

zhawan · web-flow · commit 67dea1730fba · 2022-07-05T13:42:54.000+08:00
* Update citi_bike_dashboard.py

remove missed keys in option candidates

* Update dashboard_helper.py

use a copy of attribute_option_candidates in functions
diff --git a/maro/cli/inspector/citi_bike_dashboard.py b/maro/cli/inspector/citi_bike_dashboard.py
@@ -66,7 +66,7 @@ def render_inter_view(source_path: str, epoch_num: int):
     # Get epoch sample number.
     down_pooling_range = helper._get_sampled_epoch_range(epoch_num, sample_ratio)
     attribute_option_candidates = (
-        CITIBIKEItemOption.quick_info + CITIBIKEItemOption.requirement_info + CITIBIKEItemOption.station_info
+        CITIBIKEItemOption.quick_info + CITIBIKEItemOption.requirement_info + ["bikes", "capacity"]
     )
     # Generate data.
     data_summary = helper.read_detail_csv(
diff --git a/maro/cli/inspector/dashboard_helper.py b/maro/cli/inspector/dashboard_helper.py
@@ -1,6 +1,7 @@
 # Copyright (c) Microsoft Corporation.
 # Licensed under the MIT license.
 
+import copy
 import math
 from typing import List
 
@@ -206,10 +207,10 @@ def _get_attribute_option(
     if scenario == GlobalScenarios.CITI_BIKE:
         for item in attribute_option:
             if item == "All":
-                attribute_option_candidates.remove("All")
-                attribute_option_candidates.remove("Requirement Info")
-                attribute_option_candidates.remove("Station Info")
-                attribute_option_res = attribute_option_candidates
+                attribute_option_res = copy.deepcopy(attribute_option_candidates)
+                attribute_option_res.remove("All")
+                attribute_option_res.remove("Requirement Info")
+                attribute_option_res.remove("Station Info")
                 break
             elif item == "Requirement Info":
                 attribute_option_res = attribute_option_res + CITIBIKEItemOption.requirement_info
@@ -222,10 +223,10 @@ def _get_attribute_option(
     if scenario == GlobalScenarios.CIM:
         for item in attribute_option:
             if item == "All":
-                attribute_option_candidates.remove("All")
-                attribute_option_candidates.remove("Booking Info")
-                attribute_option_candidates.remove("Port Info")
-                attribute_option_res = attribute_option_candidates
+                attribute_option_res = copy.deepcopy(attribute_option_candidates)
+                attribute_option_res.remove("All")
+                attribute_option_res.remove("Booking Info")
+                attribute_option_res.remove("Port Info")
                 break
             elif item == "Booking Info":
                 attribute_option_res = attribute_option_res + CIMItemOption.booking_info

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ def render_inter_view(source_path: str, epoch_num: int):`
`66`	`66`	`# Get epoch sample number.`
`67`	`67`	`down_pooling_range = helper._get_sampled_epoch_range(epoch_num, sample_ratio)`
`68`	`68`	`attribute_option_candidates = (`
`69`		`- CITIBIKEItemOption.quick_info + CITIBIKEItemOption.requirement_info + CITIBIKEItemOption.station_info`
	`69`	`+ CITIBIKEItemOption.quick_info + CITIBIKEItemOption.requirement_info + ["bikes", "capacity"]`
`70`	`70`	`)`
`71`	`71`	`# Generate data.`
`72`	`72`	`data_summary = helper.read_detail_csv(`