Skip to content

Downstream 1285: Add a fusion rewrite for CAReduces with Elemwise inputs #40

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

Merged
merged 6 commits into from
Nov 27, 2022

Conversation

twiecki
Copy link
Member

@twiecki twiecki commented Nov 26, 2022

Downstreaming: aesara-devs/aesara#1285

@twiecki twiecki mentioned this pull request Nov 26, 2022
Closed
@codecov-commenter
Copy link

Codecov Report

Merging #40 (b292146) into main (e15c09d) will increase coverage by 0.03%.
The diff coverage is 94.52%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   74.19%   74.22%   +0.03%     
==========================================
  Files         174      174              
  Lines       48680    48734      +54     
  Branches    10361    10367       +6     
==========================================
+ Hits        36120    36175      +55     
- Misses      10270    10272       +2     
+ Partials     2290     2287       -3     
Impacted Files Coverage Δ
pytensor/compile/function/pfunc.py 84.18% <ø> (-0.24%) ⬇️
pytensor/compile/function/types.py 79.16% <75.00%> (+0.16%) ⬆️
pytensor/tensor/elemwise.py 88.07% <90.54%> (-0.52%) ⬇️
pytensor/tensor/rewriting/elemwise.py 86.40% <94.28%> (+0.65%) ⬆️
pytensor/scalar/basic.py 79.02% <95.16%> (+0.10%) ⬆️
pytensor/compile/mode.py 84.47% <100.00%> (+1.22%) ⬆️
pytensor/tensor/math.py 90.40% <100.00%> (+0.37%) ⬆️

@twiecki twiecki requested a review from ricardoV94 November 26, 2022 23:31
@ricardoV94 ricardoV94 merged commit 1b76af3 into main Nov 27, 2022
@ricardoV94 ricardoV94 added the aesara downstream PRs that were downstreamed from Aesara label Dec 13, 2022
@ricardoV94 ricardoV94 changed the title Downstream 1285 Downstream 1285: Add a fusion rewrite for CAReduces with Elemwise inputs Feb 20, 2023
@ricardoV94 ricardoV94 deleted the downstream_1285 branch June 13, 2024 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aesara downstream PRs that were downstreamed from Aesara
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants