-
Notifications
You must be signed in to change notification settings - Fork 129
Remove unnecessary handling of no longer supported RandomState #1300
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
Remove unnecessary handling of no longer supported RandomState #1300
Conversation
Codecov ReportAttention: Patch coverage is
❌ Your patch status has failed because the patch coverage (83.33%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #1300 +/- ##
=======================================
Coverage 82.01% 82.02%
=======================================
Files 207 207
Lines 49250 49242 -8
Branches 8734 8732 -2
=======================================
- Hits 40394 40389 -5
+ Misses 6692 6690 -2
+ Partials 2164 2163 -1
🚀 New features to boost your workflow:
|
There are others at other places of the code, are those still needed?
|
702889b
to
f71e5b9
Compare
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@Armavica thanks for pointing those. I removed a couple more. I left the sharedvariable register so users get a useful error for a while longer (and the respective test). The only other non-graph use of RandomState is in |
f71e5b9
to
b7bed61
Compare
Small cleanup
📚 Documentation preview 📚: https://pytensor--1300.org.readthedocs.build/en/1300/