hard-coded #54

New Issue

2025-10-28T15:23:26+01:00

Simo93-rgb commented

2025-10-28 15:23:26 +01:00

(Migrated from github.com)

Parametri hardcoded

Ci sono parametri come il numero di articoli di NewsAgent che sono hardcoded e si rischia di avere incoerenza.

Vanno centralizzati dento a config.yaml nella apposita sezione tool

# Parametri hardcoded Ci sono parametri come il numero di articoli di NewsAgent che sono hardcoded e si rischia di avere incoerenza. Vanno centralizzati dento a `config.yaml` nella apposita sezione `tool`

Simo93-rgb commented

2025-10-28 15:26:25 +01:00

(Migrated from github.com)

Creato a mano il branch 54-hard-coded

Berack96 commented

2025-10-28 15:49:30 +01:00

(Migrated from github.com)

Io non vedo da nessuna parte che ci sono dei parametri Hard-coded per le news.
L'unico che ha un magic number è newsapi.py che ha un max_pag_size che mi sembra sia un blocco proprio della API.

Quali sono i file indicati?

Inoltre, se pubblichi il branch puoi poi collegarlo nella sezione developement qui a fianco

Io non vedo da nessuna parte che ci sono dei parametri Hard-coded per le news. L'unico che ha un magic number è [newsapi.py](https://github.com/Berack96/upo-appAI/blob/6d0b81603390ca038a732d4bd9c0afcdaa687c5a/src/app/api/news/newsapi.py#L29) che ha un max_pag_size che mi sembra sia un blocco proprio della API. Quali sono i file indicati? Inoltre, se pubblichi il branch puoi poi collegarlo nella sezione developement qui a fianco

Simo93-rgb commented

2025-10-28 16:48:27 +01:00

(Migrated from github.com)

C'è il limite a 100 per il numero di articoli. Appena finiamo una cosa pubblichiamo il branch. Lo abbiamo fatto a partire dal branch 41 e per qualche ragione da pagina web non lo permetteva. Dopo lo collego.

Berack96 commented

2025-10-28 17:17:17 +01:00

(Migrated from github.com)

Si, ma non è un limite imposto finale, è solo un valore che viene passato se non si mettono valori in limit viene scelto come default.
Inoltre, questo valore è passato dalla LLM e scelto da essa, quindi è inutile metterlo come config.

In più non fare nuovi branch da un branch in cui lavorate, ma prendete sempre il main come verità assoluta. Questo anche perchè nel main ci sono delle modifiche proprio alle config che vengono passate ai Tools

Si, ma non è un limite imposto finale, è solo un valore che viene passato se non si mettono valori in limit viene scelto come default. Inoltre, questo valore è passato dalla LLM e scelto da essa, quindi è inutile metterlo come config. In più non fare nuovi branch da un branch in cui lavorate, ma prendete sempre il main come verità assoluta. Questo anche perchè nel main ci sono delle modifiche proprio alle config che vengono passate ai Tools

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Berack/upo-app-agents#54