NoneType Error Debugging

Traceback (most recent call last): File "/Users/danielkliewer/news03/news_digest.py", line 670, in <module> main() ~~~~^^ File "/Users/danielkliewer/news03/news_digest.py", line 609, in main articles = processor.fetch_articles_enhanced(max_articles_per_feed=3) File "/Users/danielkliewer/news03/news_digest.py", line 322, in fetch_articles_enhanced article_data = self.fetch_and_process_article(entry, source_metadata) TypeError: 'NoneType' object is not callable (venv) danielkliewer@Daniels-MacBook-Pro news

The error message: ``` TypeError: 'NoneType' object is not callable ``` means that somewhere in your code, you're trying to **call something like a function**, but it's actually `None`. ### Here's how to debug this: The error is raised in this line: ```python article_data = self.fetch_and_process_article(entry, source_metadata) ``` So Python thinks `self.fetch_and_process_article` is `None`. --- ### 🔍 Check these things: 1. **Was `fetch_and_process_article` ever reassigned?** Look for a line like: ```python self.fetch_and_process_article = None ``` Or some conditional logic that might do this. 2. **Is the method defined correctly in your class?** Make sure somewhere above, in your class definition, you have something like: ```python def fetch_and_process_article(self, entry, source_metadata): ... ``` 3. **Was it accidentally overwritten?** For example, check for something like: ```python self.fetch_and_process_article = some_function_returning_none() ``` This would replace the method with `None`. --- ### ✅ Fix it: If you accidentally overwrote the method, rename the variable or remove the line that assigns `None`. If the method is missing, define it properly in your class. --- If you're still stuck, paste the class definition (especially where `fetch_and_process_article` is used or defined), and I can walk you through the exact fix.

def fetch_articles_enhanced(self, max_articles_per_feed=5): """Enhanced article fetching with database integration""" feed_urls = self.load_feeds_from_db() articles = [] print("Fetching and parsing RSS feeds with enhanced metadata...\n") for url in tqdm(feed_urls, desc="Scraping feeds", unit="feed"): domain = urlparse(url).netloc.replace('www.', '') source_metadata = self.sources_cache.get(domain) if not source_metadata: print(f"No metadata found for {domain}, skipping...") continue feed = feedparser.parse(url) for entry in feed.entries[:max_articles_per_feed]: article_data = self.fetch_and_process_article(entry, source_metadata) if article_data: articles.append(article_data) return articles

Thanks! Your `fetch_articles_enhanced` method looks fine — the issue isn't here directly, but it's where the error **manifests**. The real problem is that `self.fetch_and_process_article` is `None` **at the time it is called**. ### 🔧 Let's break this down: When you do: ```python article_data = self.fetch_and_process_article(entry, source_metadata) ``` Python thinks `self.fetch_and_process_article` is not a function but actually `None`. --- ### ✅ Action plan: #### 1. **Check your class for an assignment like this:** Search your code (maybe with Cmd+F) for anywhere you do: ```python self.fetch_and_process_article = ... ``` You're likely **overwriting the method** somewhere, like: ```python self.fetch_and_process_article = None ``` Or: ```python self.fetch_and_process_article = maybe_returning_none() ``` If you find such a line, fix it or remove it. --- #### 2. **Make sure the method is defined:** In your class (possibly called `NewsProcessor` or similar), make sure this method exists: ```python def fetch_and_process_article(self, entry, source_metadata): # process logic here return {...} ``` If it’s missing, define it. --- ### 🧠 Pro tip: print the method before calling it To verify its state, just before the call, add: ```python print("fetch_and_process_article is:", self.fetch_and_process_article) ``` If it prints: ```bash fetch_and_process_article is: None ``` Then you know for sure it's been overwritten or not initialized correctly. --- If you're still unsure, paste the relevant part of the class where `fetch_and_process_article` should be defined, or show any lines assigning it, and I’ll pinpoint the fix for you.

def fetch_and_process_article(self, entry, source_metadata): """Enhanced article fetching with metadata""" try: article = Article(entry.link) article.download() article.parse() if len(article.text) < 100: # Skip very short articles return None # Store in database with embedding embedding = self.kg.embedder.encode(article.text) story_id = self.db.store_story( title=entry.title, url=entry.link, content=article.text, source_id=source_metadata['id'], published=entry.get('published', 'N/A'), embedding=embedding ) # Add to knowledge graph self.kg.add_story_node(story_id, article.text, { 'title': entry.title, 'source': source_metadata['name'], 'published': entry.get('published', 'N/A'), 'bias_score': source_metadata['bias_score'], 'geographic_region': source_metadata['geographic_region'], 'language': source_metadata['language'] }) return { 'id': story_id, 'title': entry.title, 'link': entry.link, 'published': entry.get('published', 'N/A'), 'content': article.text[:2000], # Truncate for processing 'source_metadata': source_metadata } except Exception as e: print(f"Error processing article {entry.link}: {e}") return None

Thanks for sharing — your `fetch_and_process_article` method is properly defined. That means one of the following **must** be happening: --- ### 🧠 Root Cause Summary: **Even though the method exists**, something in your code is likely doing: ```python self.fetch_and_process_article = None ``` or: ```python self.fetch_and_process_article = some_function_that_returns_none() ``` **after** the method is defined, which **overwrites** the method with `None`. --- ### ✅ How to Find the Problem #### 1. **Search your entire class or script** for: ```python fetch_and_process_article ``` Look for any assignments like: ```python self.fetch_and_process_article = ... ``` #### 2. **Add a print/debug right before the method is called:** In `fetch_articles_enhanced`, just before the line: ```python article_data = self.fetch_and_process_article(entry, source_metadata) ``` Add: ```python print("DEBUG: fetch_and_process_article is", self.fetch_and_process_article) ``` You’ll likely see: ```python DEBUG: fetch_and_process_article is None ``` This confirms it was **overwritten**. --- ### 🧪 If you still can't find it Try printing right after class initialization: ```python processor = NewsProcessor() # or whatever your class is print("At init:", processor.fetch_and_process_article) ``` Then again just before calling `fetch_articles_enhanced`. If it changes between those two points, then something inside your code is reassigning it — possibly in your `__init__`, or right after instantiation. --- ### 🔍 Bonus tip: if you're dynamically assigning or testing methods Sometimes people do things like: ```python self.fetch_and_process_article = getattr(self, "some_optional_func", None) ``` Make sure you aren’t doing anything dynamic that might overwrite the method with `None`. --- If you'd like me to find it fast, paste your class's `__init__` method or anything that touches `fetch_and_process_article` after it's defined. I’ll spot it instantly.