emmenove / osda

0 stars 0 forks source link

Errore export risultati #1

Closed matteobalsemin closed 2 months ago

matteobalsemin commented 2 months ago

Dopo aver completato con successo l'elaborazione della lista (menù Process>Check), viene visualizzato un errore qualora si decida di esportare i risultati (menù Export>Export excel results). In allegato screenshot dell'errore.

Qualora invece si provi ad utilizzare la funzione "export raw results", la procedura viene completata con successo (vedi screenshot allegato), ma il file di output è vuoto (vedi allegato).

Errore export excel: Errore_export_excel

Conferma export raw results: Conferma_export_raw raw-results.json

emmenove commented 2 months ago

Ciao, l'errore è causato dalla mancanza di risultati. Lo noto dalla icona orologio a fianco del nome azienda in lista, che significa appunto che non esistono risultati per la riga

Correggerò l'anomalia indicando la mancanza di dati, ma al momento vorrei capire come mai non hai risultati. Mi potresti passare la schermata della configurazione ed il file relativo? (tasto window+R e digita %AppData%\osda\config.json nella casella di testo)

Di seguito cosa dovresti vedere in tabella dopo il process: image

matteobalsemin commented 2 months ago

Ciao Francesco, il file di configurazione contiene questo setup: { "apiKey": "80551709570ae74b128c4ac75a2e2057", "collezione": "sanctions", "nominativo": "Ragione sociale", "giurisdizione": "", "indirizzo": "", "limit": "5", "threshold": "0.7", "cutoff": "0.5" }

immagine

matteobalsemin commented 2 months ago

Questo il file con la lista delle società

Export 22_08_2024 16_26_VR.xlsx

emmenove commented 2 months ago

Ok, visto il problema. Ho fatto casino con la chiamata.... Intanto aggiungi in configurazione, nel campo giurisdizione il valore 'Paese' (è l'indicazione del campo da cui prendere il paese nel file) Con quello dovrebbe andare.

Domani cerco di correggere il problema Ciao

matteobalsemin commented 2 months ago

ciao Francesco, ti confermo che con il valore "Paese" funziona. Visto che questo non è un campo obbligatorio (e, in molti casi, avrò solo la "ragione sociale" senza il "paese"), puoi prevedere a codice che se il campo Paese è presente nella lista importata allora viene passato per l'elaborazione, altrimenti viene utilizzato solo la ragione sociale?

matteobalsemin commented 2 months ago

O meglio ancora, semplifichiamo le cose: se nella configurazione dell'applicativo viene popolato il campo "giurisdizione", allora utilizza il valore nella chiamata API per tutte le entry della lista. Altrimenti utilizzi solo l'info sulla ragione sociale. Sarò io ad attivare/disattivare manualmente il campo dalla configurazione per tutte le entry della lista. Fammi sapere se può aiutarti

emmenove commented 2 months ago

Ciao! Ho implementato le logiche: 1) Se definisci il campo viene utilizzato nella chiamata, a patto che sia valorizzato nel file excel 2) Se non lo definisci viene escluso per tutti (ma non si arrabbia)

Ho inoltre rilasciato le seguenti modifiche

Per il significato dei valori di configurazione verifica la pagina di help: https://api.opensanctions.org/#tag/Matching/operation/match_match__dataset__post

Trovi la nuova release all'indirizzo: https://github.com/emmenove/osda/releases/download/untagged-d7043cbc3403d81fa409/osda-0.1.2.Setup.exe

PS: chiudo questa issue e procediamo su nuove

matteobalsemin commented 2 months ago

Ciao Francesco, sei un fenomeno, volevo proprio chiederti di poter giocare con il parametro dell’algoritmo perché quello attuale non mi convince del tutto 😉

Nei prossimi giorni ci gioco un po’ e poi ti dico. Mi sto sentendo con alcuni smanettoni nella loro piattaforma Slack per farmi suggerire il setup migliore per la tipologia di screening che devo fare. Ti scrivo se dovesse servire qualche altro parametro nella chiamata

Ciao e buon weekend!