Remove pie chart for negative coefficients; set normalize to true

rsmtool/notebooks/builtin_model.ipynb

@@ -151,13 +151,13 @@
     "    ax1.set_title('Values of standardized coefficients')\n",
     "    ax1.set_xlabel('')\n",
     "    ax1.set_ylabel('')\n",
-    "    # no pie chart if we have more than 15 features or if the feature names are long (pie chart looks ugly)\n",
-    "    if len(features_used) <= 15 and longest_feature_name <= 10:\n",
+    "    # no pie chart if we have more than 15 features,\n",
+    "    # if the feature names are long (pie chart looks ugly)\n",
+    "    # or if there are any negative coefficients.\n",
+    "    if len(features_used) <= 15 and longest_feature_name <= 10 and (df_betas_sorted['relative']>=0).all():\n",
     "        ax2=fig.add_subplot(133, aspect=True)\n",
-    "        # we set normalize to False to make sure we have a partial pie in case \n",
-    "        # of negative coefficients.\n",
     "        ax2.pie(abs(df_betas_sorted['relative'].values), colors=grey_colors, \n",
-    "            labels=df_betas_sorted['feature'].values, normalize=False)\n",
+    "            labels=df_betas_sorted['feature'].values, normalize=True)\n",
     "        ax2.set_title('Proportional contribution of each feature')\n",
     "    else:\n",
     "        fig.set_size_inches(len(features_used), 3)\n",