e-mission / em-public-dashboard

A simple and stupid public dashboard prototype.
BSD 3-Clause "New" or "Revised" License
0 stars 23 forks source link

Changes to added python files to support the NREL hosted version #52

Closed shankari closed 2 years ago

shankari commented 2 years ago

Testing done:

@@ -113,7 +113,7 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "expanded_ct = scaffolding.expand_userinputs(labeled_ct)"
+    "expanded_ct = scaffolding.expand_userinputs(labeled_ct, 2)"
    ]
   },
   {
@@ -157,7 +167,17 @@
     "## Mapping new labels with dictionaries\n",
     "expanded_ct['Trip_purpose']= expanded_ct['purpose_confirm'].map(dic_pur)\n",
     "expanded_ct['Mode_confirm']= expanded_ct['mode_confirm'].map(dic_re)\n",
-    "expanded_ct['Replaced_mode']= expanded_ct['replaced_mode'].map(dic_re)"
+    "# expanded_ct['Replaced_mode']= expanded_ct['replaced_mode'].map(dic_re)"
+   ]
+  },

I can generate at least the basic mode share pie chart for the NREL commute dataset

This is a partial fix for https://github.com/e-mission/e-mission-docs/issues/781

shankari commented 2 years ago

Testing graphs:

Mode share Commute Mode Share
image image
shankari commented 2 years ago

@zackAemmer this addresses https://github.com/e-mission/e-mission-docs/issues/781#issuecomment-1230850536

zackAemmer commented 2 years ago

Thank you, should we merge this into main then I can pull the changes to my branch?

shankari commented 2 years ago

Yup. Once you approve it, I will merge into main and you can pull into your branch.