- Notifications
You must be signed in to change notification settings - Fork180
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
bugfix for tutorial on embedding net#1159
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## main #1159 +/- ##===========================================- Coverage 83.93% 73.81% -10.13%=========================================== Files 90 92 +2 Lines 6930 7259 +329 ===========================================- Hits 5817 5358 -459- Misses 1113 1901 +788
Flags with carried forward coverage won't be shown.Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Looks good to me! Thanks for contributing this small albeit important fix. :-)
See#1158