-
Notifications
You must be signed in to change notification settings - Fork 749
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
Added paper code examples as tests #123
Added paper code examples as tests #123
Conversation
Job PR-123/1 is complete. |
) | ||
|
||
|
||
def test_arxiv_example(): |
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.
The names are a bit misleading. It's probably better to have something like test_listing_1
and test_appendix_c
.
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.
Yeah, the icml and arxiv listings 1 are the same, but I guess we cannot update the ICML paper anyway once the proceedings a published. Renamed.
64919b7
Job PR-123/2 is complete. |
Codecov Report
@@ Coverage Diff @@
## master #123 +/- ##
==========================================
- Coverage 79.05% 79.01% -0.04%
==========================================
Files 110 110
Lines 6301 6305 +4
==========================================
+ Hits 4981 4982 +1
- Misses 1320 1323 +3
|
Added paper code examples from the arxiv paper as tests. If these break, we need to update the arxiv paper code examples. One caveat is that the tests do not pull and run electricity because it takes too long. So if this line changes, the example would still break. But we should be able to catch a lot of other API changes this way.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.