Skip to content

Make tests compatible with latest release of PyTensor #6782

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 1 commit into from
Jun 17, 2023

Conversation

ricardoV94
Copy link
Member

@ricardoV94 ricardoV94 commented Jun 16, 2023

Tests started failing in other PRs.

These changes work with both the previous and newer release, so there's no need to bump the dependency


📚 Documentation preview 📚: https://pymc--6782.org.readthedocs.build/en/6782/

@ricardoV94 ricardoV94 added tests pytensor no releasenotes Skipped in automatic release notes generation labels Jun 16, 2023
@ricardoV94 ricardoV94 requested a review from ferrine June 16, 2023 05:52
@codecov
Copy link

codecov bot commented Jun 16, 2023

Codecov Report

Merging #6782 (9bbe3d3) into main (9bb3cf0) will increase coverage by 14.90%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #6782       +/-   ##
===========================================
+ Coverage   74.07%   88.98%   +14.90%     
===========================================
  Files          95       95               
  Lines       16173    16173               
===========================================
+ Hits        11980    14391     +2411     
+ Misses       4193     1782     -2411     
Impacted Files Coverage Δ
pymc/model.py 89.91% <ø> (+20.04%) ⬆️

... and 43 files with indirect coverage changes

@twiecki twiecki merged commit 2806c23 into pymc-devs:main Jun 17, 2023
@ricardoV94 ricardoV94 deleted the pytensor_compat branch September 18, 2023 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no releasenotes Skipped in automatic release notes generation pytensor tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants