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

Create a base type for shared RNG variables #1476

Merged

Conversation

brandonwillard
Copy link
Member

This PR creates a base type for shared RNG variables (i.e. a base class for RandomStateSharedVariable and RandomGeneratorSharedVariable) and fills in some type hints for SharedVariable itself.

@brandonwillard brandonwillard added enhancement New feature or request random variables Involves random variables and/or sampling graph objects typing Work related to type checking labels Mar 16, 2023
@brandonwillard brandonwillard self-assigned this Mar 16, 2023
@brandonwillard brandonwillard enabled auto-merge (rebase) March 16, 2023 22:32
@codecov
Copy link

codecov bot commented Mar 16, 2023

Codecov Report

Merging #1476 (7fa7024) into main (462d8d5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1476   +/-   ##
=======================================
  Coverage   75.02%   75.02%           
=======================================
  Files         194      194           
  Lines       50099    50102    +3     
  Branches    12096    12096           
=======================================
+ Hits        37586    37589    +3     
  Misses      10189    10189           
  Partials     2324     2324           
Impacted Files Coverage Δ
aesara/compile/sharedvalue.py 93.75% <100.00%> (ø)
aesara/tensor/random/var.py 96.00% <100.00%> (+0.54%) ⬆️

@brandonwillard brandonwillard merged commit b75eed9 into aesara-devs:main Mar 17, 2023
@brandonwillard brandonwillard deleted the create-base-shared-rng-type branch March 17, 2023 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request graph objects random variables Involves random variables and/or sampling typing Work related to type checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant