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

Add support for shared inputs in numba_funcify_Scan #1241

Merged

Conversation

brandonwillard
Copy link
Member

This PR adds missing shared input support to numba_funcify_Scan and fixes a Type check bug in make_numba_random_fn.

@brandonwillard brandonwillard marked this pull request as draft October 6, 2022 14:58
@brandonwillard brandonwillard added Numba Involves Numba transpilation Scan Involves the `Scan` `Op` random variables Involves random variables and/or sampling labels Oct 6, 2022
@brandonwillard brandonwillard force-pushed the add-Numba-Scan-shared-inputs branch 2 times, most recently from ffd9ff7 to 1cf82f7 Compare October 7, 2022 06:55
@brandonwillard brandonwillard marked this pull request as ready for review October 7, 2022 06:56
@brandonwillard brandonwillard self-assigned this Oct 7, 2022
@codecov
Copy link

codecov bot commented Oct 7, 2022

Codecov Report

Merging #1241 (bc73460) into main (a619a4e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1241   +/-   ##
=======================================
  Coverage   79.14%   79.15%           
=======================================
  Files         173      173           
  Lines       48528    48536    +8     
  Branches    10322    10322           
=======================================
+ Hits        38408    38417    +9     
  Misses       7628     7628           
+ Partials     2492     2491    -1     
Impacted Files Coverage Δ
aesara/link/numba/dispatch/random.py 98.81% <100.00%> (-0.01%) ⬇️
aesara/link/numba/dispatch/scan.py 95.31% <100.00%> (+1.19%) ⬆️

@brandonwillard brandonwillard merged commit 739bd49 into aesara-devs:main Oct 7, 2022
@brandonwillard brandonwillard deleted the add-Numba-Scan-shared-inputs branch October 7, 2022 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Numba Involves Numba transpilation random variables Involves random variables and/or sampling Scan Involves the `Scan` `Op`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant