fixed request implementation

app.py

@@ -27,53 +27,54 @@ def paper_finder(topics:list, max_paper:int)-> str: #it's import to specify the
         
         try:
             response = requests.get(url)
-            if response.status_code == 200:
-                # parsing element tree as text
-                root = ET.from_string(response.text)
-
-                # extract paper information
-                papers = []
-                
-                # arxiv returns responses in Atom 1.0 format "https://info.arxiv.org/help/api/user-manual.html#32-the-api-response"
-                namespace = {"atom": 'http://www.w3.org/2005/Atom'}
-
-                # list of the returned Atom results
-                for entry in root.findall('atom:entry', namespace):
-                    title = entry.find('atom:title', namespace).text.strip()
-                    summary = entry.find('atom:summary', namespace).text.strip()
-                    published = entry.find('atom:published', namespace).text.strip()[:10]
-                    link = entry.find('./atom:link[@title="pdf"]', namespace)
-                    if link is not None:
-                        pdf_url = link.get('href')
-                    else:
-                        pdf_url = "No PDF link available"
-
-                    # Authors of the paper
-                    authors = []
-                    for author in entry.findall('atom:author/atom:name', namespace):
-                        authors.append(author.text.strip())
-    
-                    # Adding paper
-                    papers.append({
-                        "title": title,
-                        "authors": ", ".join(authors[:3]) + ("..." if len(authors) > 3 else ""),
-                        "published": published,
-                        "summary": summary[:150] + "..." if len(summary) > 150 else summary,
-                        "pdf_url": pdf_url
-                    })
-    
-                    for i, paper in enumerate(papers, 1):
-                        topic_results += f"{i}. **{paper['title']}**\n"
-                        topic_results += f"   *Authors:* {paper['authors']}\n"
-                        topic_results += f"   *Published:* {paper['published']}\n"
-                        topic_results += f"   *Summary:* {paper['summary']}\n"
-                        topic_results += f"   *PDF:* {paper['pdf_url']}\n\n"
+            response.raise_for_status()
             
-                    results.append(topic_results)
-        
-            else:
-                print(f"An error occured when requesting for {query}, please try again!")
-        
+            # parsing element tree as text
+            root = ET.from_string(response.text)
+
+            # extract paper information
+            papers = []
+            
+            # arxiv returns responses in Atom 1.0 format "https://info.arxiv.org/help/api/user-manual.html#32-the-api-response"
+            namespace = {"atom": 'http://www.w3.org/2005/Atom'}
+
+            # list of the returned Atom results
+            for entry in root.findall('atom:entry', namespace):
+                title = entry.find('atom:title', namespace).text.strip()
+                summary = entry.find('atom:summary', namespace).text.strip()
+                published = entry.find('atom:published', namespace).text.strip()[:10]
+                link = entry.find('./atom:link[@title="pdf"]', namespace)
+                if link is not None:
+                    pdf_url = link.get('href')
+                else:
+                    pdf_url = "No PDF link available"
+
+                # Authors of the paper
+                authors = []
+                for author in entry.findall('atom:author/atom:name', namespace):
+                    authors.append(author.text.strip())
+
+                # Adding paper
+                papers.append({
+                    "title": title,
+                    "authors": ", ".join(authors[:3]) + ("..." if len(authors) > 3 else ""),
+                    "published": published,
+                    "summary": summary[:150] + "..." if len(summary) > 150 else summary,
+                    "pdf_url": pdf_url
+                })
+
+            # Format the results for this topic
+            topic_results = f"\n## Latest papers on {topic} ({len(papers)})\n\n"
+
+            for i, paper in enumerate(papers, 1):
+                topic_results += f"{i}. **{paper['title']}**\n"
+                topic_results += f"   *Authors:* {paper['authors']}\n"
+                topic_results += f"   *Published:* {paper['published']}\n"
+                topic_results += f"   *Summary:* {paper['summary']}\n"
+                topic_results += f"   *PDF:* {paper['pdf_url']}\n\n"
+
+            results.append(topic_results)
+    
         except Exception as e:
             results.append(f"\n## Error searching for {topic}: {str(e)}\n")