Aggregazione Market Product non corretta #62

New Issue

Berack · 2025-10-29T21:16:45+01:00

Berack96 commented

2025-10-29 21:16:45 +01:00

(Migrated from github.com)

La funzione di aggregazione non aggrega correttamente i dati, dato che se si passano due cripto con due valute diverse, la funzione tende ad aggeregarle in ogni caso.

Checklist

Aggiungere dei test di aggregazione che controllino questi casi limite
Mettere a posto l'aggregazione sull'id del prodotto (classe Product) invece che solo sul simbolo come è ora markets.py
Modificare il product_id aggregated in modo che restituisca correttamente l'id + la stringa "_AGGREGATED"
Controllare ed eventualmente aggiornare anche la classe Price sempre dentro markets.py
Controllare che la media sia corretta dal punti di vista dell'economia (?) ovvero se è corretta come è fatta ora.
Esportare, invece che una lista dei prezzi aggregati, due liste (dentro una tupla) una con gli Exchange e una con i prezzi.

La funzione di aggregazione non aggrega correttamente i dati, dato che se si passano due cripto con due valute diverse, la funzione tende ad aggeregarle in ogni caso. ## Checklist - [ ] Aggiungere dei test di aggregazione che controllino questi casi limite - [x] Mettere a posto l'aggregazione sull'id del prodotto (classe Product) invece che solo sul simbolo come è ora [markets.py](https://github.com/Berack96/upo-appAI/blob/main/src/app/api/core/markets.py) - [x] Modificare il product_id aggregated in modo che restituisca correttamente l'id + la stringa "_AGGREGATED" - [ ] Controllare ed eventualmente aggiornare anche la classe [Price](https://github.com/Berack96/upo-appAI/blob/ff6f422fe1887f3b1a1c78a44cabcc9455f59668/src/app/api/core/markets.py#L71) sempre dentro markets.py - [x] Controllare che la media sia corretta dal punti di vista dell'economia (?) ovvero se è corretta come è fatta ora. - [ ] Esportare, invece che una lista dei prezzi aggregati, due liste (dentro una tupla) una con gli Exchange e una con i prezzi.

Simo93-rgb commented

2025-10-30 15:37:22 +01:00

(Migrated from github.com)

Ho notato che non chiama mai una valuta diversa dal dollaro e robabilmente era un caso sfortunato quello che il prezzo allucinato era una via di mezzo fra euro e dollaro. Ho cambiato l'aggregazione e ora se per caso riceve una valuta differente fra i vari exchange lui di default prende solo il dollaro. Ho fatto fare dei test in più. Ho aggiunto un tool in più per permettere di prendere una sola crypto da più exchange.

Berack closed this issue

2026-01-18 12:50:49 +01:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Berack/upo-app-agents#62