Skip to content

[SECURITY] Replace 14 bare except blocks in pt_hub.py - closes #53 #91

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
sjackson0109 merged 4 commits into from
May 21, 2026
Merged

[SECURITY] Replace 14 bare except blocks in pt_hub.py - closes #53 #91

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c6a90aa
fix: Replace 14 bare except blocks in pt_hub.py with specific types
Ibrahim-3d May 18, 2026
e37c21d
fix: Black formatting on pt_hub.py
Ibrahim-3d May 18, 2026
060088c
style: apply Black formatting to pt_hub.py
Ibrahim-3d May 19, 2026
1d330c6
Merge branch 'main' into feat/53-bare-except-audit
sjackson0109 May 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 14 additions & 14 deletions app/pt_hub.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4247,7 +4247,7 @@ def _create_llm_research_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating LLM Research tab: {e}")
import traceback
Expand Down Expand Up @@ -4288,7 +4288,7 @@ def _create_holdings_management_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Holdings Management tab: {e}")
import traceback
Expand Down Expand Up @@ -4329,7 +4329,7 @@ def _create_portfolio_analytics_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Portfolio Analytics tab: {e}")
import traceback
Expand Down Expand Up @@ -4370,7 +4370,7 @@ def _create_advanced_order_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Advanced Order Types tab: {e}")
import traceback
Expand Down Expand Up @@ -4411,7 +4411,7 @@ def _create_market_data_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Real-time Market Data tab: {e}")
import traceback
Expand Down Expand Up @@ -4455,7 +4455,7 @@ def _create_portfolio_optimization_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Portfolio Optimization tab: {e}")
import traceback
Expand Down Expand Up @@ -4496,7 +4496,7 @@ def _create_backtesting_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Backtesting Framework tab: {e}")
import traceback
Expand Down Expand Up @@ -4541,7 +4541,7 @@ def _create_performance_attribution_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Performance Attribution tab: {e}")
import traceback
Expand Down Expand Up @@ -4582,7 +4582,7 @@ def _create_institutional_trading_tab(self):
font=("TkDefaultFont", 10),
anchor="center",
).pack(expand=True, fill="both", padx=20, pady=20)
except:
except Exception:
pass
print(f"Error creating Institutional Trading tab: {e}")
import traceback
Expand Down Expand Up @@ -5494,7 +5494,7 @@ def start_trainer_for_selected_coin(self) -> None:
print(f"DEBUG: Process output: {stdout}")
if stderr:
print(f"DEBUG: Process stderr: {stderr}")
except:
except (TimeoutError, OSError):
pass
return # Don't register a failed process

Expand Down Expand Up @@ -5543,7 +5543,7 @@ def _monitor_trainer_process(self, coin: str) -> None:
print(
f"DEBUG: Final output from {coin}: {remaining_output}"
)
except:
except (OSError, AttributeError):
pass
else:
# Process still running, check again in 2 seconds
Expand Down Expand Up @@ -5626,7 +5626,7 @@ def _create_crypto_icon_grid(
# Ensure we get 3 coins in normal view, 4-5 in fullscreen
if max_cols < 3:
max_cols = 3
except:
except Exception:
max_cols = 3 # Default to 3 for better wrapping

row = 0
Expand Down Expand Up @@ -5715,7 +5715,7 @@ def on_enter(event):
frame.configure(relief="raised", bd=3)
else:
frame.configure(relief="raised", bd=3)
except:
except Exception:
frame.configure(relief="raised", bd=3)

def on_leave(event):
Expand Down Expand Up @@ -5852,7 +5852,7 @@ def on_coin_click(event=None):
"Training Error",
f"Failed {action} training for {coin_name}: {e}",
)
except:
except Exception:
pass

return on_coin_click
Expand Down
Loading