Skip to content
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

Implement several RandomVariables as SymbolicRandomVariables #7239

Merged
merged 3 commits into from
Apr 14, 2024

Commits on Apr 10, 2024

  1. Remove wrong instantiation from test

    This is not only not used anywhere, but could also try to initiate an RV with incompatible shapes and size
    ricardoV94 committed Apr 10, 2024
    Configuration menu
    Copy the full SHA
    8d637d9 View commit details
    Browse the repository at this point in the history

Commits on Apr 12, 2024

  1. Tweaks to SymbolicRandomVariables

    * Allow signature to handle rng and size arguments explicitly.
    * Parse ndim_supp and ndims_params from class signature
    * Move rv_op method to the SymbolicRandomVariable class and get rid of dummy inputs logic (it was needed in previous versions of PyTensor)
    * Fix errors in automatic signature of CustomDist
    * Allow dispatch methods without filtering of inputs for SymbolicRandomVariable distributions
    ricardoV94 committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    9d5f066 View commit details
    Browse the repository at this point in the history
  2. Reimplement several RandomVariables as SymbolicRandomVariables

    This allows sampling from multiple backends without having to dispatch for each one
    ricardoV94 committed Apr 12, 2024
    Configuration menu
    Copy the full SHA
    597e195 View commit details
    Browse the repository at this point in the history