reduced font news headlines

app.py

@@ -133,16 +133,16 @@ class NewsSentiment:
     def get_sentiment_and_headlines(self, symbol):
         """
         Try NewsAPI first, fallback to Alpha Vantage if needed.
-        Returns (sentiment, headlines, source).
+        Returns (sentiment, headlines, source, avg_score).
         """
         # Try NewsAPI
         try:
             articles = self.newsapi.get_everything(q=symbol, language='en', sort_by='publishedAt', page=1)
             headlines = [a['title'] for a in articles.get('articles', [])[:5]]
             if headlines:
-                sentiment = self._calculate_sentiment(headlines)
+                sentiment, avg_score = self._calculate_sentiment(headlines, return_score=True)
                 logger.info(f"NewsAPI sentiment for {symbol}: {sentiment} | Headlines: {headlines}")
-                return sentiment, headlines, "NewsAPI"
+                return sentiment, headlines, "NewsAPI", avg_score
             else:
                 logger.warning(f"NewsAPI returned no headlines for {symbol}.")
         except Exception as e:
@@ -153,7 +153,7 @@ class NewsSentiment:
         try:
             if not self.alpha_vantage_api_key:
                 logger.error("Alpha Vantage API key not found in Streamlit secrets.")
-                return None, [], "AlphaVantage"
+                return None, [], "AlphaVantage", None
             import requests
             url = (
                 f"https://www.alphavantage.co/query?function=NEWS_SENTIMENT&tickers={symbol}"
@@ -163,9 +163,9 @@ class NewsSentiment:
             data = resp.json()
             headlines = [item.get("title") for item in data.get("feed", [])[:5] if item.get("title")]
             if headlines:
-                sentiment = self._calculate_sentiment(headlines)
+                sentiment, avg_score = self._calculate_sentiment(headlines, return_score=True)
                 logger.info(f"Alpha Vantage sentiment for {symbol}: {sentiment} | Headlines: {headlines}")
-                return sentiment, headlines, "AlphaVantage"
+                return sentiment, headlines, "AlphaVantage", avg_score
             else:
                 logger.warning(f"Alpha Vantage returned no headlines for {symbol}.")
         except Exception as e:
@@ -174,19 +174,20 @@ class NewsSentiment:
         logger.info(
             f"No sentiment/headlines available for {symbol} from either NewsAPI or Alpha Vantage."
         )
-        return None, [], None
+        return None, [], None, None
 
-    def _calculate_sentiment(self, headlines):
+    def _calculate_sentiment(self, headlines, return_score=False):
         if not headlines:
-            return None
+            return (None, None) if return_score else None
         compound_score = sum(self.sia.polarity_scores(title)['compound'] for title in headlines)
         avg_score = compound_score / len(headlines)
         if avg_score > 0.1:
-            return 'Positive'
+            sentiment = 'Positive'
         elif avg_score < -0.1:
-            return 'Negative'
+            sentiment = 'Negative'
         else:
-            return 'Neutral'
+            sentiment = 'Neutral'
+        return (sentiment, avg_score) if return_score else sentiment
 
     def get_sentiment_bulk(self, symbols):
         """
@@ -414,39 +415,45 @@ class TradingApp:
                 st.session_state["article_headlines"] = []
             if "sentiment_source" not in st.session_state:
                 st.session_state["sentiment_source"] = None
+            if "sentiment_score" not in st.session_state:
+                st.session_state["sentiment_score"] = None
 
             if st.button("Check Sentiment"):
                 if symbol:
-                    sentiment_result, article_headlines, sentiment_source = self.sentiment.get_sentiment_and_headlines(symbol)
+                    sentiment_result, article_headlines, sentiment_source, sentiment_score = self.sentiment.get_sentiment_and_headlines(symbol)
                     st.session_state["sentiment_result"] = sentiment_result
                     st.session_state["article_headlines"] = article_headlines
                     st.session_state["sentiment_symbol"] = symbol
                     st.session_state["sentiment_source"] = sentiment_source
+                    st.session_state["sentiment_score"] = sentiment_score
                 else:
                     st.session_state["sentiment_result"] = None
                     st.session_state["article_headlines"] = []
                     st.session_state["sentiment_symbol"] = ""
                     st.session_state["sentiment_source"] = None
+                    st.session_state["sentiment_score"] = None
 
             sentiment_result = st.session_state.get("sentiment_result")
             article_headlines = st.session_state.get("article_headlines", [])
             sentiment_symbol = st.session_state.get("sentiment_symbol", "")
             sentiment_source = st.session_state.get("sentiment_source", "")
+            sentiment_score = st.session_state.get("sentiment_score", None)
 
             if symbol and sentiment_symbol == symbol and sentiment_result is not None:
                 st.markdown(f"**Sentiment for {symbol.upper()} ({sentiment_source}):** {sentiment_result if sentiment_result in ['Positive', 'Negative', 'Neutral'] else 'No sentiment available'}")
+                # Show the actual average score below
+                if sentiment_score is not None:
+                    st.markdown(f"<span style='font-size:0.8em;color:#888;'>Average sentiment score: <b>{sentiment_score:.3f}</b></span>", unsafe_allow_html=True)
             elif symbol and sentiment_symbol == symbol and sentiment_result is None:
                 st.markdown("**Sentiment:** No sentiment available")
 
-            # Shrink headlines font and remove bullets, add clickable links
+            # Shrink headlines font, number them, and use dark blue/white with underline for links
             if symbol and sentiment_symbol == symbol and article_headlines:
                 st.markdown(
-                    "<div style='font-size: 0.85em; margin-bottom: 0.5em;'><b>Recent Headlines:</b></div>",
+                    "<div style='font-size: 0.75em; margin-bottom: 0.5em;'><b>Recent Headlines:</b></div>",
                     unsafe_allow_html=True
                 )
                 # Try to get URLs for headlines if available
-                # Fetch the latest headlines and URLs from the same source as get_sentiment_and_headlines
-                # For NewsAPI, get 'url' from articles; for Alpha Vantage, get 'url' from feed
                 headlines_with_links = []
                 try:
                     if sentiment_source == "NewsAPI":
@@ -468,21 +475,26 @@ class TradingApp:
                             (item.get("title"), item.get("url")) for item in feed if item.get("title")
                         ]
                     else:
-                        # fallback: just show headlines without links
                         headlines_with_links = [(headline, None) for headline in article_headlines]
                 except Exception as e:
                     logger.error(f"Error fetching URLs for headlines: {e}")
                     headlines_with_links = [(headline, None) for headline in article_headlines]
 
-                for headline, url in headlines_with_links:
+                # Use dark blue (#003366) for links, white for dark backgrounds (auto-detect not possible, so use blue)
+                for idx, (headline, url) in enumerate(headlines_with_links, 1):
                     if url:
                         st.markdown(
-                            f"<div style='font-size: 0.85em; margin-bottom: 0.2em; color: #444;'><a href='{url}' target='_blank' style='color: #444; text-decoration: underline;'>{headline}</a></div>",
+                            f"<div style='font-size:0.75em; margin-bottom:0.15em; color:#003366;'>"
+                            f"<span style='font-weight:bold;'>{idx}.</span> "
+                            f"<a href='{url}' target='_blank' style='color:#003366; text-decoration:underline;'>{headline}</a>"
+                            f"</div>",
                             unsafe_allow_html=True
                         )
                     else:
                         st.markdown(
-                            f"<div style='font-size: 0.85em; margin-bottom: 0.2em; color: #444;'>{headline}</div>",
+                            f"<div style='font-size:0.75em; margin-bottom:0.15em; color:#003366;'>"
+                            f"<span style='font-weight:bold;'>{idx}.</span> {headline}"
+                            f"</div>",
                             unsafe_allow_html=True
                         )
             elif symbol and sentiment_symbol == symbol and sentiment_result is not None and not article_headlines:
@@ -658,7 +670,7 @@ def get_market_times(alpaca_api):
 
 def main():
     st.title("Ben's Stock Trading Application")
-    st.markdown("This is a fun stock trading application that uses a combination of key frameworks like Alpaca API, yfinance, and News API for stock information and trading. Come and trade my money! Well, it's a paper account, so it's not real money. But still, have fun!")
+    st.markdown("This is a fun stock trading application that uses a combination of key frameworks like Alpaca API, yfinance, News API, and Alpha Vantage for stock information and trading. Come and trade my money! Well, it's a paper account, so it's not real money. But still, have fun!")
 
     if not st.secrets['ALPACA_API_KEY'] or not st.secrets['NEWS_API_KEY']:
         st.error("Please configure your ALPACA_API_KEY and NEWS_API_KEY")