Berack/upo-app-agents
0
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Lista di cryptovalute #36

Merged
Berack96 merged 9 commits from 31-lista-di-cryptovalute into main 2025-10-21 15:58:42 +02:00
Conversation 2 Commits 9 Files Changed 13 +33 -2
1 changed files with 33 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d94c616fa3 - Show all commits

35
src/app/api/crypto_symbols.py → src/app/api/tools/symbols_tool.py
View File

@@ -4,14 +4,16 @@ import asyncio
import logging import logging
import pandas as pd import pandas as pd
from io import StringIO from io import StringIO
from agno.tools.toolkit import Toolkit
logging.basicConfig(level=logging.INFO) logging.basicConfig(level=logging.INFO)
logging = logging.getLogger("crypto_symbols") logging = logging.getLogger("crypto_symbols")
BASE_URL = "https://finance.yahoo.com/markets/crypto/all/" BASE_URL = "https://finance.yahoo.com/markets/crypto/all/"
class CryptoSymbols: class CryptoSymbols(Toolkit):
""" """
Classe per ottenere i simboli delle criptovalute tramite Yahoo Finance. Classe per ottenere i simboli delle criptovalute tramite Yahoo Finance.
""" """
@@ -19,11 +21,40 @@ class CryptoSymbols:
def __init__(self, cache_file: str = 'cryptos.csv'): def __init__(self, cache_file: str = 'cryptos.csv'):
self.cache_file = cache_file self.cache_file = cache_file
self.final_table = pd.read_csv(self.cache_file) if os.path.exists(self.cache_file) else pd.DataFrame() # type: ignore self.final_table = pd.read_csv(self.cache_file) if os.path.exists(self.cache_file) else pd.DataFrame() # type: ignore
copilot-pull-request-reviewer[bot] commented 2025-10-21 13:23:20 +02:00 (Migrated from github.com)
Review
Copy Link

[nitpick] This line is overly complex with multiple operations. Consider splitting it into separate lines for better readability: check file existence, then read CSV or create empty DataFrame.

        if os.path.exists(self.cache_file):
            self.final_table = pd.read_csv(self.cache_file) # type: ignore
        else:
            self.final_table = pd.DataFrame() # type: ignore
[nitpick] This line is overly complex with multiple operations. Consider splitting it into separate lines for better readability: check file existence, then read CSV or create empty DataFrame. ```suggestion if os.path.exists(self.cache_file): self.final_table = pd.read_csv(self.cache_file) # type: ignore else: self.final_table = pd.DataFrame() # type: ignore ```
Toolkit.__init__(self, # type: ignore
name="Crypto Symbols Tool",
tools=[
self.get_all_symbols,
self.get_symbol_by_name,
],
)
def get_symbols(self) -> list[str]: def get_all_symbols(self) -> list[str]:
"""
Restituisce tutti i simboli delle criptovalute.
Returns:
list[str]: Lista di tutti i simboli delle criptovalute.
"""
return self.final_table['Symbol'].tolist() if not self.final_table.empty else [] return self.final_table['Symbol'].tolist() if not self.final_table.empty else []
def get_symbol_by_name(self, query: str) -> list[str]:
"""
Cerca i simboli che contengono la query.
Args:
query (str): Query di ricerca.
Returns:
list[str]: Lista di simboli che contengono la query.
"""
query_lower = query.lower()
positions = self.final_table['Name'].str.lower().str.contains(query_lower)
return self.final_table[positions]['Symbol'].tolist()
async def fetch_crypto_symbols(self, force_refresh: bool = False) -> None: async def fetch_crypto_symbols(self, force_refresh: bool = False) -> None:
"""
Recupera tutti i simboli delle criptovalute da Yahoo Finance e li memorizza in cache.
Args:
force_refresh (bool): Se True, forza il recupero anche se i dati sono già in cache.
"""
if not force_refresh and not self.final_table.empty: if not force_refresh and not self.final_table.empty:
return return