8000 Negative Multinomial aggiornata by Davide-Ruffini · Pull Request #12 · gpitt71/gemact-code · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Negative Multinomial aggiornata #12

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 22, 2025
Merged

Conversation

Davide-Ruffini
Copy link
Contributor

Ciao a tutti,

ho aggiornato la Negative Multinomial seguendo l'implementazione di wikipedia "https://en.wikipedia.org/wiki/Negative_multinomial_distribution". In passato incontrai dei problemi sull'implementazione del metodo RVS (usare il metodo solito tramite cdf non mi risultava semplicissimo essendo una multivariata, ma magari mi sono arenato io ed esiste una formula una via semplice), che mi portò a cambiare il sistema di parametrizzazione seguendo quando implementato in R nel pacchetto MGLM, ora dovrei aver trovato una soluzione che dovrebbe funzionare:

Potremmo vedere la nostra distribuzione Negative Multinomial usando:
-Una binomiale negative per generare il numero totale
-Una multinomiale per ripartire i conteggi

In questo modo mi sembra proprio che i momenti della distribuzione simulata si parlino con quelli teorici, e l'implementazione è semplice. Fatemi sapere cosa ne pensate

Vi allego due semplici test, nel Test1 verifico che funzioni tutto a dovere.
Nel Test2 invece ho fatto qualche confronto con l'implementazione di MGLM. In effetti esiste qualche legame tra le due parametrizzazioni e si parlano quando il parametro di dispersione è 1 (formula non dispersa).

Test 2- confronto con MGLM.txt
Test1.txt

@gpitt71
Copy link
Owner
gpitt71 commented May 9, 2025

Ciao sembra che tutto sia okay adesso! Cosa ne pensi @EdoLu ?

@EdoLu
Copy link
Collaborator
EdoLu commented May 10, 2025

Ciao! Direi che va tutto bene.
@Davide-Ruffini grazie per quanto hai fatto. Ti direi come ultima cosa di modificare nelle nuove distribuzioni multivariate che hai creato il metodo statico "category" come segue

@staticmethod
def category():
return {''}

in modo che non ritorni 'frequency', in quanto, al momento, non possono essere usate come componente Frequency nella classe LossModel. @gpitt71 concordi?

Edoardo

@Davide-Ruffini
Copy link
Contributor Author

Ciao a tutti.
Ok chiarissimo, aspetto che Gabri ti risponda e procedo
Davide

@EdoLu
Copy link
Collaborator
EdoLu commented May 16, 2025

@gpitt71 puoi dare il nullaosta a @Davide-Ruffini ?

@gpitt71
Copy link
Owner
gpitt71 commented May 19, 2025

Ciao a tutti, sono d'accordo

categories of multivariate distributions updated
@Davide-Ruffini
Copy link
Contributor Author

Ciao a tutti! Ho applicato la modifica alle distribuzioni multivariate

Davide

@gpitt71
Copy link
Owner
gpitt71 commented May 20, 2025

Se siamo tutti allineati possiamo fare il merge. @EdoLu

@EdoLu
Copy link
Collaborator
EdoLu commented May 22, 2025 via email

@gpitt71 gpitt71 merged commit bbf140c into gpitt71:main May 22, 2025
1 check passed
@gpitt71
Copy link
Owner
gpitt71 commented May 22, 2025

Merge completato. Grazie a tutti per il lavoro. Chiudo questa thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0