Skip to content

add icdf + remove ruff#60

Merged
lockwo merged 2 commits into
mainfrom
Owen/icdf
Mar 26, 2026
Merged

add icdf + remove ruff#60
lockwo merged 2 commits into
mainfrom
Owen/icdf

Conversation

@lockwo

@lockwo lockwo commented Mar 23, 2026

Copy link
Copy Markdown
Owner

addresses #59 and #58

@lockwo

lockwo commented Mar 23, 2026

Copy link
Copy Markdown
Owner Author

check this out @gvcallen if you get a chance

@gvcallen

Copy link
Copy Markdown
Contributor

Looking good to me. Thanks!

@lockwo lockwo merged commit 94d5c5c into main Mar 26, 2026
3 checks passed
@lockwo lockwo deleted the Owen/icdf branch March 26, 2026 17:19
@gvcallen

Copy link
Copy Markdown
Contributor

@lockwo do you plan on publishing this release to PyPi?

@lockwo

lockwo commented Mar 31, 2026

Copy link
Copy Markdown
Owner Author

Yes, once I get some more features in. Probably at the end of this week?

@gvcallen

gvcallen commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

Great, no rush.

Slightly off topic then, we also have some bijectors and an empirical distribution that we are using in ParamRF. Not sure if you'd be interested in integrating them, or having me submit a PR before the release?

The bijectors are of particular interest, since I couldn't find Identity, Inverse or Exp in distreqx but I feel these are rather fundamental

@lockwo

lockwo commented Mar 31, 2026

Copy link
Copy Markdown
Owner Author

Yea some of them you can see on the list to add #5, but there are definitely plenty more. I think probably most are welcome in distreqx as long as they aren't too bespoke or complicated.

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.

2 participants