diff --git a/openbb_terminal/stocks/insider/openinsider_model.py b/openbb_terminal/stocks/insider/openinsider_model.py index c494bfafd2e..f2b6a7bf246 100644 --- a/openbb_terminal/stocks/insider/openinsider_model.py +++ b/openbb_terminal/stocks/insider/openinsider_model.py @@ -1143,7 +1143,11 @@ def get_open_insider_link(preset_loaded: str) -> str: """ preset = configparser.RawConfigParser() preset.optionxform = str # type: ignore - preset.read(get_preset_choices()[preset_loaded]) + choices = get_preset_choices() + if preset_loaded not in choices: + console.print("[red]Could not find the link[/red]\n") + return "" + preset.read(choices[preset_loaded]) d_general = dict(preset["General"]) d_date = dict(preset["Date"]) diff --git a/openbb_terminal/stocks/options/options_controller.py b/openbb_terminal/stocks/options/options_controller.py index d188e8ec24d..151eb569968 100644 --- a/openbb_terminal/stocks/options/options_controller.py +++ b/openbb_terminal/stocks/options/options_controller.py @@ -1075,7 +1075,7 @@ class OptionsController(BaseController): "--x_axis", type=str, dest="x", - default=None, + default="s", choices=self.plot_vars_choices, help=( "ltd- last trade date, s- strike, lp- last price, b- bid, a- ask," @@ -1087,7 +1087,7 @@ class OptionsController(BaseController): "--y_axis", type=str, dest="y", - default=None, + default="y", choices=self.plot_vars_choices, help=( "ltd- last trade date, s- strike, lp- last price, b- bid, a- ask," diff --git a/openbb_terminal/stocks/options/screen/syncretism_model.py b/openbb_terminal/stocks/options/screen/syncretism_model.py index 413eff35592..55b54e92a72 100644 --- a/openbb_terminal/stocks/options/screen/syncretism_model.py +++ b/openbb_terminal/stocks/options/screen/syncretism_model.py @@ -179,7 +179,10 @@ def get_screener_output(preset: str) -> Tuple[pd.DataFrame, str]: preset_filter = configparser.RawConfigParser() preset_filter.optionxform = str # type: ignore - preset_filter.read(get_preset_choices()[preset]) + choices = get_preset_choices() + if preset not in choices: + return pd.DataFrame(), "No data found" + preset_filter.read(choices[preset]) d_filters = {k: v for k, v in dict(preset_filter["FILTER"]).items() if v} s_filters = str(d_filters) diff --git a/openbb_terminal/stocks/options/yfinance_view.py b/openbb_terminal/stocks/options/yfinance_view.py index e98f645e166..f9607510401 100644 --- a/openbb_terminal/stocks/options/yfinance_view.py +++ b/openbb_terminal/stocks/options/yfinance_view.py @@ -696,6 +696,12 @@ def plot_plot( x = "strike" y = "impliedVolatility" else: + if x is None: + console.print("[red]Invalid option sent for x-axis[/red]\n") + return + if y is None: + console.print("[red]Invalid option sent for y-axis[/red]\n") + return x = convert[x] y = convert[y] varis = op_helpers.opt_chain_cols diff --git a/openbb_terminal/stocks/screener/screener_controller.py b/openbb_terminal/stocks/screener/screener_controller.py index 5c5b368da20..77c45407c4f 100644 --- a/openbb_terminal/stocks/screener/screener_controller.py +++ b/openbb_terminal/stocks/screener/screener_controller.py @@ -167,8 +167,9 @@ class ScreenerController(BaseController): if other_args and "-" not in other_args[0][0]: other_args.insert(0, "-p") ns_parser = self.parse_known_args_and_warn(parser, other_args) - screener_view.display_presets(ns_parser.preset) - console.print("") + if ns_parser: + screener_view.display_presets(ns_parser.preset) + console.print("") @log_start_end(log=logger) def call_set(self, other_args: List[str]):