-
Notifications
You must be signed in to change notification settings - Fork 17
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
Use the doctest module in get_example_data #308
Merged
Merged
Changes from 36 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
7b6ec43
First pass at using the doctest module in get_example_data
asmeurer 8e11f52
Fix matplotlib showing the plots instead of saving them in doctests
asmeurer 3dd390b
Move the PapyriDocTestRunner class to the module level
asmeurer 0bbbabe
Handle filename and lineno not being accessible
asmeurer d65f57c
Fix a spelling error
asmeurer 471d729
Fix generation of doctest plot figures
asmeurer bbcb40e
Remove unused variable
asmeurer 29c66ac
Use ELLIPSIS option flag
asmeurer a13f7e4
First pass at using the doctest module in get_example_data
asmeurer 9cdb1ae
Fix matplotlib showing the plots instead of saving them in doctests
asmeurer 0fba48d
Move the PapyriDocTestRunner class to the module level
asmeurer 24389a4
Handle filename and lineno not being accessible
asmeurer d946cc6
Fix a spelling error
asmeurer 877026f
Fix generation of doctest plot figures
asmeurer cbd7fc0
Remove unused variable
asmeurer 83be405
Use ELLIPSIS option flag
asmeurer 1a0caee
add debug print
Carreau 98304d2
debug log
Carreau 236c309
cleanup
Carreau 841ac12
... to >>>
Carreau ef6780e
debug and execute config
Carreau 507e37c
don't execute in pandas
Carreau cfbacb7
Add Section.extend
asmeurer 3e2409d
Split example sections into blocks
asmeurer 8f6c1da
Merge branch 'doctest' of github.com:asmeurer/papyri into doctest
asmeurer a2da2d8
Fix errors from running doctests on SciPy
asmeurer 6da0c62
Merge branch 'main' into doctest
asmeurer 9bf10fb
Fix the --no-exec flag
asmeurer a1c8cdb
Excluse `set_numeric_ops` which seem to be the root of crash.
Carreau db1dbb4
reforamt to please linters
Carreau 7ce945e
Merge branch 'main' into doctest
asmeurer 4d114ff
Fix the doctest runner not maintaining the namespace across different…
asmeurer 4100b5e
Use DocTestParser to parse doctest blocks
asmeurer dea1888
Don't generate text blocks for empty strings
asmeurer 708dac9
Fix duplicate code entries in example section
asmeurer 96502c7
Fix extending figures.
Carreau eb498cb
Apply suggestions from code review
Carreau 4621cba
Rename config.exec to execute_doctests.
Carreau 45a2fa3
Rename config.exec to execute_doctests.
Carreau 81071b9
rename exec in a few more places
Carreau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
You introduced this but are not using it, I'm guessing you intended something to
exec
. I pushed a commit that rename all the usage ofconfig.exec
toconfig.execute_doctests
for it to work as intended. You naming is better.