changed naming conventions for easier reading

app.py

@@ -49,12 +49,12 @@ def show_all_stats(see_full_stats):
         return pd.DataFrame()
 
 
-def fixChoicesCorrectOrder(choices):
-    ListChoices = [x for x in Columns if x in choices]
-    return ListChoices
+def fix_choices_correct_order(choices):
+    list_choices = [x for x in Columns if x in choices]
+    return list_choices
 
 
-def MatchNameToUser(user_names):
+def match_name_lower_case(user_names):
     users = []
     for N in user_names:
         users.append(UserNameDict[N])
@@ -69,9 +69,9 @@ def convert_back_plot_choices(plot_choices_raw):
     return [PLOT_CHOICES_REVERSE_DICT[choice] for choice in plot_choices_raw]
 
 
-def main(From,
-         To,
-         UserNameChoices,
+def main(from_date,
+         to_date,
+         usr_name_choices,
          plot_choice,
          save_selected,
          rb1, rb2, rb3, rb4, rb5, rb6, rb7, rb8,
@@ -106,10 +106,11 @@ def main(From,
         return pie_charts
 
     def bar(db: pd.DataFrame, title):
-        '''This method adds a stacked bar diagram for each target and each sentiment
+        """
+        This method adds a stacked bar diagram for each target and each sentiment
         NOTE:  The tweets without any target are not shown in the plot, we just show distribution of tweets that have a
         target.
-        '''
+        """
         if db.empty:
             return None
         else:
@@ -161,7 +162,7 @@ def main(From,
             return None
         else:
             # db = db[col_name].value_counts()[:5] # Lägg till "Others sedan"
-            db = piechart_input(db, col_name, LIMIT)
+            db = pie_chart_input(db, col_name, LIMIT)
             labels = db[col_name].to_list()
             sizes = db['frequency'].values
             # explode = (0, 0.1, 0, 0, 0)  # only "explode" the 2nd slice (i.e. 'Hogs')
@@ -171,7 +172,7 @@ def main(From,
             plt.title(title, fontdict=font1)
             return fig
 
-    text_classifier = tc.TextClassifier(from_date=From, to_date=To, user_list=MatchNameToUser(UserNameChoices),
+    text_classifier = tc.TextClassifier(from_date=from_date, to_date=to_date, user_list=match_name_lower_case(usr_name_choices),
                                         num_tweets=num_tweet)
     text_classifier.run_main_pipeline()
     dataframe = text_classifier.get_dataframe()
@@ -180,7 +181,7 @@ def main(From,
     df = dataframe
 
     if save_selected:
-        user_list = MatchNameToUser(UserNameChoices)
+        user_list = match_name_lower_case(usr_name_choices)
         df_l = []
         for user in user_list:
             df_l.append(pd.DataFrame(df.loc[df['username'] == user]))
@@ -192,7 +193,7 @@ def main(From,
     else:
         save_selected_checkbox = [gr.Checkbox.update(interactive=True)]
 
-    pycharts = add_pie_chart(df, UserNameChoices, convert_plot_choices(plot_choice))
+    pycharts = add_pie_chart(df, usr_name_choices, convert_plot_choices(plot_choice))
 
     rb_components = [rb1, rb2, rb3, rb4, rb5, rb6, rb7, rb8]  # radio_buttons
     df_visibility_check = [v1, v2, v3, v4, v5, v6, v7, v8]
@@ -214,13 +215,13 @@ def main(From,
                 save_file_components_list.append(gr.Checkbox.update(visible=u_bool))
 
             if u_bool and visibility[i]:
-                df_list.append(get_exemple_df(user_df, PLOT_CHOICES_DICT[radio[i]]))
+                df_list.append(get_example_df(user_df, PLOT_CHOICES_DICT[radio[i]]))
             else:
                 df_list.append(None)
 
         return df_list + number_tweets + save_file_components_list
 
-    return pycharts + save_selected_checkbox + get_selected_df_list(df, save_file_bool, list(UserNameChoices),
+    return pycharts + save_selected_checkbox + get_selected_df_list(df, save_file_bool, list(usr_name_choices),
                                                                     rb_components, df_visibility_check)
 
 
@@ -232,7 +233,7 @@ def main(From,
 '''
 
 
-def get_exemple_df(df: pd.DataFrame, column: str):
+def get_example_df(df: pd.DataFrame, column: str):
     print(column)
     df = df[SELECTED_COLUMN_DICT[column]]
     unique_labels = df[column].value_counts().keys()
@@ -243,11 +244,11 @@ def get_exemple_df(df: pd.DataFrame, column: str):
             df_temp = df_temp[0:5]
         stat.append(df_temp)
 
-    exemple_df = pd.concat(stat)
+    example_df = pd.concat(stat)
 
     # stat =stat.reset_index(drop=True)     just in case u want to reset indexing
 
-    return exemple_df
+    return example_df
 
 
 def export_to_download(_data_frame, _type: str):
@@ -272,7 +273,7 @@ def export_to_download(_data_frame, _type: str):
     # , pie_chart(df, "main_topic"), pie_chart("target")
 
 
-def piechart_input(df, column, limit):
+def pie_chart_input(df, column, limit):
     df_len = len(df)
     df_v = df[column].value_counts()
     df_len = len(df)
@@ -285,9 +286,7 @@ def piechart_input(df, column, limit):
         df_v = ds_target.value_counts()
         freq = df_v.to_list()
         labels = df_v.keys().to_list
-        freq_dict = {}
-        freq_dict[column] = labels
-        freq_dict["frequency"] = freq
+        freq_dict = {column: labels, "frequency": freq}
         return pd.DataFrame.from_dict(freq_dict)
 
     else:
@@ -326,7 +325,7 @@ def update_window(leaders: list, plot_choices: list,
     leader_bool_list, plot_bool_list = convert_to_boolean(leaders, plot_choices)
 
     bool_list = []
-    df_visiblity_bool = [v1, v2, v3, v4, v5, v6, v7, v8]
+    df_visibility_bool = [v1, v2, v3, v4, v5, v6, v7, v8]
 
     # this loop sets boolean for plots
     for leader in leader_bool_list:
@@ -353,8 +352,8 @@ def update_window(leaders: list, plot_choices: list,
         update_blocks.append(gr.Row.update(visible=vis_or_not))
         update_blocks.append(gr.Row.update(visible=vis_or_not))
         if vis_or_not:
-            update_blocks.append(gr.Row.update(visible=df_visiblity_bool[i]))
-            update_df.append(gr.DataFrame.update(visible=df_visiblity_bool[i]))
+            update_blocks.append(gr.Row.update(visible=df_visibility_bool[i]))
+            update_df.append(gr.DataFrame.update(visible=df_visibility_bool[i]))
         else:
 
             update_blocks.append(gr.Row.update(visible=False))
@@ -390,8 +389,8 @@ if __name__ == "__main__":
             with gr.Row():
                 with gr.Column():
                     with gr.Row():
-                        date1 = gr.Textbox(label="From", value='2022-05-10')
-                        date2 = gr.Textbox(label="To", value='2022-05-30')
+                        date1 = gr.Textbox(label="from_date", value='2022-05-10')
+                        date2 = gr.Textbox(label="to_date", value='2022-05-30')
                     leaders = gr.Checkboxgroup(choices=USER_NAMES,
                                                label="")
                     plot_choices = gr.CheckboxGroup(choices=CHOICE_LIST, label='Choose what to show')