From 855f211efa40589cde83ac7ebfb430cf9eb20d83 Mon Sep 17 00:00:00 2001 From: Colin Delahunty <72827203+colin99d@users.noreply.github.com> Date: Tue, 9 Aug 2022 15:16:33 -0400 Subject: [PATCH] Fixed glitch (#2280) --- .../stocks/insider/insider_controller.py | 2 +- .../stocks/insider/openinsider_view.py | 41 +++++++++++-------- 2 files changed, 24 insertions(+), 19 deletions(-) diff --git a/openbb_terminal/stocks/insider/insider_controller.py b/openbb_terminal/stocks/insider/insider_controller.py index 38cda4e7753..2acb6c39f53 100644 --- a/openbb_terminal/stocks/insider/insider_controller.py +++ b/openbb_terminal/stocks/insider/insider_controller.py @@ -283,7 +283,7 @@ class InsiderController(StockBaseController): add_help=False, formatter_class=argparse.ArgumentDefaultsHelpFormatter, prog="stats", - description="Print open insider filtered data using selected ticker. [Source: OpenInsider]", + description="Open insider filtered data using selected ticker. [Source: OpenInsider]", ) parser.add_argument( "-l", diff --git a/openbb_terminal/stocks/insider/openinsider_view.py b/openbb_terminal/stocks/insider/openinsider_view.py index 2ba46339bf7..cc4a68d0c22 100644 --- a/openbb_terminal/stocks/insider/openinsider_view.py +++ b/openbb_terminal/stocks/insider/openinsider_view.py @@ -264,37 +264,42 @@ def print_insider_filter( ).head(limit) if rich_config.USE_COLOR and not links: - if not df_insider[df_insider["Trade Type"] == "S - Sale"].empty: - df_insider[df_insider["Trade Type"] == "S - Sale"] = df_insider[ - df_insider["Trade Type"] == "S - Sale" + new_df_insider = df_insider.copy() + if not new_df_insider[new_df_insider["Trade Type"] == "S - Sale"].empty: + new_df_insider[new_df_insider["Trade Type"] == "S - Sale"] = new_df_insider[ + new_df_insider["Trade Type"] == "S - Sale" ].apply(lambda_red_highlight) - if not df_insider[df_insider["Trade Type"] == "S - Sale+OE"].empty: - df_insider[df_insider["Trade Type"] == "S - Sale+OE"] = df_insider[ - df_insider["Trade Type"] == "S - Sale+OE" - ].apply(lambda_yellow_highlight) - if not df_insider[df_insider["Trade Type"] == "F - Tax"].empty: - df_insider[df_insider["Trade Type"] == "F - Tax"] = df_insider[ - df_insider["Trade Type"] == "F - Tax" + if not new_df_insider[new_df_insider["Trade Type"] == "S - Sale+OE"].empty: + new_df_insider[ + new_df_insider["Trade Type"] == "S - Sale+OE" + ] = new_df_insider[new_df_insider["Trade Type"] == "S - Sale+OE"].apply( + lambda_yellow_highlight + ) + if not new_df_insider[new_df_insider["Trade Type"] == "F - Tax"].empty: + new_df_insider[new_df_insider["Trade Type"] == "F - Tax"] = new_df_insider[ + new_df_insider["Trade Type"] == "F - Tax" ].apply(lambda_magenta_highlight) - if not df_insider[df_insider["Trade Type"] == "P - Purchase"].empty: - df_insider[df_insider["Trade Type"] == "P - Purchase"] = df_insider[ - df_insider["Trade Type"] == "P - Purchase" - ].apply(lambda_green_highlight) + if not new_df_insider[new_df_insider["Trade Type"] == "P - Purchase"].empty: + new_df_insider[ + new_df_insider["Trade Type"] == "P - Purchase" + ] = new_df_insider[new_df_insider["Trade Type"] == "P - Purchase"].apply( + lambda_green_highlight + ) patch_pandas_text_adjustment() pd.set_option("display.max_colwidth", 0) pd.set_option("display.max_rows", None) # needs to be done because table is too large :( - df_insider = df_insider.drop(columns=["Filing Date", "Trade Type"]) + new_df_insider = new_df_insider.drop(columns=["Filing Date", "Trade Type"]) else: # needs to be done because table is too large :( - df_insider = df_insider.drop(columns=["Filing Date"]) + new_df_insider = df_insider.drop(columns=["Filing Date"]) print_rich_table( - df_insider, - headers=[x.title() for x in df_insider.columns], + new_df_insider, + headers=[x.title() for x in new_df_insider.columns], title="Insider filtered", )