Skip to content

feat: Dagum distribution #2014

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 18 commits into from
Jun 3, 2025
Merged

feat: Dagum distribution #2014

merged 18 commits into from
Jun 3, 2025

Conversation

Qazalbash
Copy link
Contributor

Summary

This PR contains the implementation of the Dagum distribution, along with the unit tests and documentation.

@Qazalbash
Copy link
Contributor Author

The stuck test case in modeling passes locally on both py3.9 and py3.10.

@fehiepsi
Copy link
Member

Thanks @Qazalbash! Sorry for the late response. I think you can revise some parameters in T(dist.Dagum, ...) to have lower variance.

@fehiepsi
Copy link
Member

could you merge with the master branch? the failing tests have been fixed there

@fehiepsi
Copy link
Member

fehiepsi commented Jun 2, 2025

Just curious, why did you remove mean/var? is it unstable or something?

@Qazalbash
Copy link
Contributor Author

Yes, they were unstable! I couldn't figure out why the test cases for them were failing.

@fehiepsi
Copy link
Member

fehiepsi commented Jun 2, 2025

Could you add them back? I can take a look.

@Qazalbash
Copy link
Contributor Author

Ok I am reverting the commit.

@fehiepsi fehiepsi merged commit 7a3c24f into pyro-ppl:master Jun 3, 2025
9 checks passed
@Qazalbash Qazalbash deleted the dagum-dist branch June 3, 2025 20:40
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy