-
Notifications
You must be signed in to change notification settings - Fork 218
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
Support non-ASCII characters in function arguments #2584
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
75778bf
Support non-ASCII characters
seisman e840c99
Support all ISOlatin1 characters
seisman 1b6d940
Support more ISOLatin1+ characters
seisman 1b88cd6
fix
seisman 311b976
Update pygmt/helpers/utils.py
seisman 672413b
Refactor to make it more readable
seisman 2dcc288
Need to remove single quote
seisman c9b8254
[ci skip] Use a better reference for ASCII table
seisman e2947fa
Support Symbols charset
seisman 1a84634
Support ZapfDingbats charset
seisman 636ace0
Refactor and add more doctests
seisman ffabaee
Fix a symbol which is incorrectly copied from PDF
seisman e1b43b2
Replace octal codes with non-ASCII character in two examples
seisman 7ea78da
Fix a typo in doctest
seisman 487f2d8
Merge branch 'main' into non-ascii-support
seisman 288486c
Fix some characters
seisman 97a223d
Fix symbol characters
seisman 4ff2e56
Add one more reference
seisman 388109f
Add two more references
seisman 2270d9d
Update ZapfDingbats charset
seisman 117b6e5
Make it clear that Symbol/ZapfDingbats are from Adobe
seisman 1798ccc
Update for ISOLatin1+ charset
seisman e4bedb9
Fix registered sign, copyright sign and trade mark sign
seisman 5ef84f9
Add more notes
seisman 490535c
Add a test for non-ascii support
seisman 418adc5
Update the dvc file
seisman 4b66b8a
Fix styling issue
seisman 37b0c6a
Add docstrings
seisman 3362b31
Update examples/gallery/embellishments/colorbar.py
seisman 695c59b
Merge branch 'main' into non-ascii-support
seisman bbc223b
Remove an unused pylint directive
seisman 35306bf
Fix a typo in doctest
seisman e56359e
Merge branch 'main' into non-ascii-support
seisman 633b2f9
Merge branch 'main' into non-ascii-support
seisman cdb4ab1
Merge branch 'main' into non-ascii-support
seisman 646465c
Update pygmt/helpers/utils.py
seisman 9ce0217
Merge branch 'main' into non-ascii-support
seisman 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
outs: | ||
- md5: d93bed7495d77cd2ef7cc1b64edb9b3a | ||
size: 19563 | ||
path: test_non_ascii_to_octal.png |
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.
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.
Symbols \140, \275 and \276 don't exist in the unicode table, so they're incorrectly shown.
Some symbols of \36x and \37x are shown as boxes in the GitHub web GUI, but they're shown correctly in Vim, so these symbols should be good.