-
-
Notifications
You must be signed in to change notification settings - Fork 612
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
Adding initialstates function to RNNs #2541
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2541 +/- ##
===========================================
+ Coverage 32.69% 65.81% +33.12%
===========================================
Files 34 35 +1
Lines 1976 2030 +54
===========================================
+ Hits 646 1336 +690
+ Misses 1330 694 -636 ☔ View full report in Codecov by Sentry. |
It should be defined also for |
Ah sorry I was meaning to ask, I'll go forward with that |
Should test that the |
Adding
initialstates
function as per discussion in #2514PR Checklist
Entry in NEWS.md