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

Fix warning in ReplaceValide.replace_all_validate_remove #1042

Merged

Conversation

brandonwillard
Copy link
Member

This PR fixes a broken warning in ReplaceValide.replace_all_validate_remove.

@codecov
Copy link

codecov bot commented Jul 9, 2022

Codecov Report

Merging #1042 (375f528) into main (510a961) will increase coverage by 0.00%.
The diff coverage is 50.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1042   +/-   ##
=======================================
  Coverage   79.26%   79.26%           
=======================================
  Files         152      152           
  Lines       47873    47874    +1     
  Branches    10908    10908           
=======================================
+ Hits        37947    37948    +1     
  Misses       7429     7429           
  Partials     2497     2497           
Impacted Files Coverage Δ
aesara/graph/features.py 64.69% <50.00%> (+0.09%) ⬆️

@brandonwillard brandonwillard added bug Something isn't working graph rewriting labels Jul 10, 2022
@brandonwillard brandonwillard merged commit c1fef51 into aesara-devs:main Jul 10, 2022
@brandonwillard brandonwillard deleted the fix-ReplaceValide-warning branch July 10, 2022 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working graph rewriting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants