-
Notifications
You must be signed in to change notification settings - Fork 71
Add ShiftedBetaGeometric
discrete distribution
#574
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
Closed
Closed
Changes from 52 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
269dd75
dist and rv init commit
ColtAllen b264161
Merge branch 'pymc-devs:main' into grassia2geo-dist
ColtAllen d734c68
docstrings
ColtAllen 71bd632
Merge branch 'grassia2geo-dist' of https://github.com/ColtAllen/pymc-…
ColtAllen 48e93f3
Merge branch 'pymc-devs:main' into grassia2geo-dist
ColtAllen 93c4a60
unit tests
ColtAllen d2e72b5
alpha min value
ColtAllen 8685005
revert alpha lim
ColtAllen 026f182
small lam value tests
ColtAllen d12dd0b
ruff formatting
ColtAllen bcd9cac
TODOs
ColtAllen 78be107
WIP add covar support to RV
ColtAllen f3ae359
Merge branch 'main' into grassia2geo-dist
ColtAllen 8a30459
WIP time indexing
ColtAllen 7c7afc8
WIP time indexing
ColtAllen fa9c1ec
Merge branch 'grassia2geo-dist' of https://github.com/ColtAllen/pymc-…
ColtAllen b957333
WIP symbolic indexing
ColtAllen d0c1d98
delete test_simple.py
ColtAllen 264c55e
fix symbolic indexing errors
ColtAllen 05e7c55
Merge branch 'pymc-devs:main' into grassia2geo-dist
ColtAllen 0fa3390
clean up cursor code
ColtAllen 5baa6f7
warn for ndims deprecation
ColtAllen a715ec7
clean up comments and final TODO
ColtAllen f3c0f29
remove ndims deprecation and extraneous code
ColtAllen a232e4c
revert changes to irrelevant test
ColtAllen ffc059f
remove time_covariate_vector default args
ColtAllen 1d41eb7
revert remaining changes in irrelevant tests
ColtAllen 47ad523
remove test_sampling_consistency
ColtAllen 5b77263
checkpoint commit for log_cdf and test frameworks
ColtAllen eb7222f
checkpoint commit for log_cdf and test frameworks
ColtAllen b34e3d8
make C_t external function, code cleanup
ColtAllen 9803321
rng_fn cleanup
ColtAllen 5ff6853
WIP test frameworks
ColtAllen 63a0b10
inverse cdf
ColtAllen 932a046
covariate pos constraint and WIP RV
ColtAllen b78a5c4
Merge branch 'pymc-devs:main' into grassia2geo-dist
ColtAllen ab45a9c
WIP rng_fn testing
ColtAllen 0d1dcea
WIP time covars required param
ColtAllen 434e5a5
C_t for RV time covar support
ColtAllen c66c8a6
time_covar optional param
ColtAllen fb96220
restore GPT5 code
ColtAllen a0ed4f5
Merge branch 'main' into grassia2geo-dist
ColtAllen c9f5dc2
init commit, WIP testing
ColtAllen 217c521
TODOs and WIP logp testing
ColtAllen d39ec3e
WIP recursive logp
ColtAllen cd7815a
revert to beta logp
ColtAllen 4f56c2a
remove logcdf
ColtAllen 3fd1cc7
docstrings
ColtAllen 11a8cac
docstrings plot code
ColtAllen d5a98c6
WIP test log_p
ColtAllen e61b702
Merge branch 'pymc-devs:main' into sbg-dist
ColtAllen e3743a3
cleanup tests
ColtAllen 3d44733
add logcdf
ColtAllen 12d2c36
SymbolicRandomVariable
ColtAllen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use a SymbolicRandomVariable that defines the random method in terms of pytensor operations. That way it will work automatically in all backends (jax/numba), not just C/python
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here is an example: https://github.com/pymc-devs/pymc/blob/9f653a6ebcc4a205b1f506e2f3e513753d718b28/pymc/distributions/discrete.py#L393-L412
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This might get moved into
pymc-marketing
instead, but can we resolve theSymbolicRandomVariable
change first? I started working on this but some tests are failing with the same error: