René's URL Explorer Experiment


Title: Fix wrong error message in #11919 by davidleejy · Pull Request #11930 · matplotlib/matplotlib · GitHub

Open Graph Title: Fix wrong error message in #11919 by davidleejy · Pull Request #11930 · matplotlib/matplotlib

X Title: Fix wrong error message in #11919 by davidleejy · Pull Request #11930 · matplotlib/matplotlib

Description: PR Summary Fixes issue #11919 . This PR chains (appropriate) exceptions raised by mcolors.to_rgba_array() to additional exceptions raised by its caller scatter(). Now, the exception raised by mcolors.to_rgba_array() is visible if color values are outside of 0 to 1 range. Prior to this PR, exceptions are not chained - see commit e3a8004. Exceptions raised by mcolors.to_rgba_array() were suppressed by exceptions of its caller scatter(). This PR applies exception chaining solely on scatter function. Author is open to helping chain other exceptions. PEP 3134 advises on Exception Chaining. Example use: https://stackoverflow.com/questions/16414744/python-exception-chaining Outcome Case 1: color values outside 0 to 1 range python -c "import numpy as np; import matplotlib.pyplot as plt; f =plt.figure(); ax = plt.axes(); ax.scatter(np.array([2]), np.array([3]), c=np.array([[12,130,140]]))" Traceback (most recent call last): File "", line 1, in File "/home/d/code/dljy-matplotlib/lib/matplotlib/__init__.py", line 1764, in inner return func(ax, *args, **kwargs) File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4232, in scatter raise e File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4210, in scatter colors = mcolors.to_rgba_array(c) File "/home/d/code/dljy-matplotlib/lib/matplotlib/colors.py", line 244, in to_rgba_array raise ValueError("RGBA values should be within 0-1 range") ValueError: RGBA values should be within 0-1 range Case 2: color array is wrong shape python -c "import numpy as np; import matplotlib.pyplot as plt; f =plt.figure(); ax = plt.axes(); ax.scatter(np.array([2]), np.array([3]), c=np.array([[.12,.130,.140],[.1,.1,.1]]))" Traceback (most recent call last): File "", line 1, in File "/home/d/code/dljy-matplotlib/lib/matplotlib/__init__.py", line 1764, in inner return func(ax, *args, **kwargs) File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4232, in scatter raise e File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4220, in scatter .format(nc=n_elem, xs=x.size, ys=y.size) ValueError: 'c' argument has 2 elements, which is not acceptable for use with 'x' with size 1, 'y' with size 1. Case 3: color is 'jaune' python -c "import numpy as np; import matplotlib.pyplot as plt; f =plt.figure(); ax = plt.axes(); ax.scatter(np.array([2]), np.array([3]), c='jaune')" Traceback (most recent call last): File "/home/d/code/dljy-matplotlib/lib/matplotlib/colors.py", line 158, in to_rgba rgba = _colors_full_map.cache[c, alpha] KeyError: ('j', None) During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4210, in scatter colors = mcolors.to_rgba_array(c) File "/home/d/code/dljy-matplotlib/lib/matplotlib/colors.py", line 259, in to_rgba_array result[i] = to_rgba(cc, alpha) File "/home/d/code/dljy-matplotlib/lib/matplotlib/colors.py", line 160, in to_rgba rgba = _to_rgba_no_colorcycle(c, alpha) File "/home/d/code/dljy-matplotlib/lib/matplotlib/colors.py", line 204, in _to_rgba_no_colorcycle raise ValueError("Invalid RGBA argument: {!r}".format(orig_c)) ValueError: Invalid RGBA argument: 'j' The above exception was the direct cause of the following exception: Traceback (most recent call last): File "", line 1, in File "/home/d/code/dljy-matplotlib/lib/matplotlib/__init__.py", line 1764, in inner return func(ax, *args, **kwargs) File "/home/d/code/dljy-matplotlib/lib/matplotlib/axes/_axes.py", line 4231, in scatter ) from e ValueError: 'c' argument must either be valid as mpl color(s) or as numbers to be mapped to colors. Here c = jaune. PR Checklist Has Pytest style unit tests Code is Flake 8 compliant New features are documented, with examples if plot related Documentation is sphinx and numpydoc compliant Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there) Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

Open Graph Description: PR Summary Fixes issue #11919 . This PR chains (appropriate) exceptions raised by mcolors.to_rgba_array() to additional exceptions raised by its caller scatter(). Now, the exception raised by mcol...

X Description: PR Summary Fixes issue #11919 . This PR chains (appropriate) exceptions raised by mcolors.to_rgba_array() to additional exceptions raised by its caller scatter(). Now, the exception raised by mcol...

Opengraph URL: https://github.com/matplotlib/matplotlib/pull/11930

X: @github

direct link

Domain: github.com

route-pattern/:user_id/:repository/pull/:id/files(.:format)
route-controllerpull_requests
route-actionfiles
fetch-noncev2:0e87f01a-2dc2-2431-efa0-1536d8db97db
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-id89A0:31D771:30563:41718:6A5674F9
html-safe-nonce0d71a129386129c1c2d2afe2f5f327c818e9272ef423e441f26d3872864da49d
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4OUEwOjMxRDc3MTozMDU2Mzo0MTcxODo2QTU2NzRGOSIsInZpc2l0b3JfaWQiOiIzNDk2MzA0MzM3OTUyODYzNDgxIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0=
visitor-hmac31627a9ea6c0bb4091d8ba932da6f69c8142bafba941e2d5e83a9a90fb6e7f00
hovercard-subject-tagpull_request:210700155
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/files
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/matplotlib/matplotlib/pull/11930/files
twitter:imagehttps://avatars.githubusercontent.com/u/1690072?s=400&v=4
twitter:cardsummary_large_image
og:imagehttps://avatars.githubusercontent.com/u/1690072?s=400&v=4
og:image:altPR Summary Fixes issue #11919 . This PR chains (appropriate) exceptions raised by mcolors.to_rgba_array() to additional exceptions raised by its caller scatter(). Now, the exception raised by mcol...
og:site_nameGitHub
og:typeobject
hostnamegithub.com
expected-hostnamegithub.com
None19b5ee9431dcfce15ab4b001d626e4e3a6ba1acb7be814813a52bbebd9a86ef5
turbo-cache-controlno-preview
diff-viewunified
go-importgithub.com/matplotlib/matplotlib git https://github.com/matplotlib/matplotlib.git
octolytics-dimension-user_id215947
octolytics-dimension-user_loginmatplotlib
octolytics-dimension-repository_id1385122
octolytics-dimension-repository_nwomatplotlib/matplotlib
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id1385122
octolytics-dimension-repository_network_root_nwomatplotlib/matplotlib
turbo-body-classeslogged-out env-production page-responsive full-width
disable-turbotrue
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
releaseafc01ffdb045f4ff81c6122091366ade7b6166b7
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/matplotlib/matplotlib/pull/11930/files#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fmatplotlib%2Fmatplotlib%2Fpull%2F11930%2Ffiles
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/open-source/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/open-source/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fmatplotlib%2Fmatplotlib%2Fpull%2F11930%2Ffiles
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Ffiles&source=header-repo&source_repo=matplotlib%2Fmatplotlib
Reloadhttps://github.com/matplotlib/matplotlib/pull/11930/files
Reloadhttps://github.com/matplotlib/matplotlib/pull/11930/files
Reloadhttps://github.com/matplotlib/matplotlib/pull/11930/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/11930/files
matplotlib https://github.com/matplotlib
matplotlibhttps://github.com/matplotlib/matplotlib
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/11930/files
Notifications https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Fork 8.4k https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Star 23k https://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib
Code https://github.com/matplotlib/matplotlib
Issues 1.1k https://github.com/matplotlib/matplotlib/issues
Pull requests 409 https://github.com/matplotlib/matplotlib/pulls
Actions https://github.com/matplotlib/matplotlib/actions
Projects https://github.com/matplotlib/matplotlib/projects
Wiki https://github.com/matplotlib/matplotlib/wiki
Security and quality 0 https://github.com/matplotlib/matplotlib/security
Insights https://github.com/matplotlib/matplotlib/pulse
Code https://github.com/matplotlib/matplotlib
Issues https://github.com/matplotlib/matplotlib/issues
Pull requests https://github.com/matplotlib/matplotlib/pulls
Actions https://github.com/matplotlib/matplotlib/actions
Projects https://github.com/matplotlib/matplotlib/projects
Wiki https://github.com/matplotlib/matplotlib/wiki
Security and quality https://github.com/matplotlib/matplotlib/security
Insights https://github.com/matplotlib/matplotlib/pulse
Sign up for GitHub https://github.com/signup?return_to=%2Fmatplotlib%2Fmatplotlib%2Fissues%2Fnew%2Fchoose
terms of servicehttps://docs.github.com/terms
privacy statementhttps://docs.github.com/privacy
Sign inhttps://github.com/login?return_to=%2Fmatplotlib%2Fmatplotlib%2Fissues%2Fnew%2Fchoose
davidleejyhttps://github.com/davidleejy
matplotlib:masterhttps://github.com/matplotlib/matplotlib/tree/master
davidleejy:err-msg-fixhttps://github.com/davidleejy/matplotlib/tree/err-msg-fix
Conversation 7 https://github.com/matplotlib/matplotlib/pull/11930
Commits 4 https://github.com/matplotlib/matplotlib/pull/11930/commits
Checks 0 https://github.com/matplotlib/matplotlib/pull/11930/checks
Files changed https://github.com/matplotlib/matplotlib/pull/11930/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/11930/files
Fix wrong error message in #11919 https://github.com/matplotlib/matplotlib/pull/11930/files#top
Show all changes 4 commits https://github.com/matplotlib/matplotlib/pull/11930/files
952ca40 chained exceptions so that exceptions raised by mcolors.to_rgba_array… davidleejy Aug 24, 2018 https://github.com/matplotlib/matplotlib/pull/11930/commits/952ca40d0c17ee7ffb96a4b1cac1cd5d1969ac41
32f4492 Merge remote-tracking branch 'upstream/master' into err-msg-fix davidleejy Aug 27, 2018 https://github.com/matplotlib/matplotlib/pull/11930/commits/32f44923d76a7ba0692fb1ced1cffe28d29a3c9c
5fca75e Fix tests davidleejy Aug 27, 2018 https://github.com/matplotlib/matplotlib/pull/11930/commits/5fca75e90cf10afc91a4c2a32164111a89b0206a
9806c1a add tests for color values 0 to 255 range davidleejy Aug 27, 2018 https://github.com/matplotlib/matplotlib/pull/11930/commits/9806c1a540790a0f5f9ac44f45f0bb56e2f631cc
Clear filters https://github.com/matplotlib/matplotlib/pull/11930/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/11930/files
Please reload this pagehttps://github.com/matplotlib/matplotlib/pull/11930/files
_axes.py https://github.com/matplotlib/matplotlib/pull/11930/files#diff-608e2a1bf3d32fd631b0763599a3334d59822c11cb4fec1fa7398c9714592900
test_axes.py https://github.com/matplotlib/matplotlib/pull/11930/files#diff-ef5144bd27a700deb760ef56ccf51dc6fcfe86723f9ffa84df7f024985bf3ee7
lib/matplotlib/axes/_axes.pyhttps://github.com/matplotlib/matplotlib/pull/11930/files#diff-608e2a1bf3d32fd631b0763599a3334d59822c11cb4fec1fa7398c9714592900
View file https://github.com/davidleejy/matplotlib/blob/9806c1a540790a0f5f9ac44f45f0bb56e2f631cc/lib/matplotlib/axes/_axes.py
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/matplotlib/matplotlib/pull/11930/{{ revealButtonHref }}
https://github.com/matplotlib/matplotlib/pull/11930/files#diff-608e2a1bf3d32fd631b0763599a3334d59822c11cb4fec1fa7398c9714592900
https://github.com/matplotlib/matplotlib/pull/11930/files#diff-608e2a1bf3d32fd631b0763599a3334d59822c11cb4fec1fa7398c9714592900
lib/matplotlib/tests/test_axes.pyhttps://github.com/matplotlib/matplotlib/pull/11930/files#diff-ef5144bd27a700deb760ef56ccf51dc6fcfe86723f9ffa84df7f024985bf3ee7
View file https://github.com/davidleejy/matplotlib/blob/9806c1a540790a0f5f9ac44f45f0bb56e2f631cc/lib/matplotlib/tests/test_axes.py
Open in desktop https://desktop.github.com
https://github.co/hiddenchars
https://github.com/matplotlib/matplotlib/pull/11930/{{ revealButtonHref }}
https://github.com/matplotlib/matplotlib/pull/11930/files#diff-ef5144bd27a700deb760ef56ccf51dc6fcfe86723f9ffa84df7f024985bf3ee7
https://github.com/matplotlib/matplotlib/pull/11930/files#diff-ef5144bd27a700deb760ef56ccf51dc6fcfe86723f9ffa84df7f024985bf3ee7
https://github.com/matplotlib/matplotlib/pull/11930/files#diff-ef5144bd27a700deb760ef56ccf51dc6fcfe86723f9ffa84df7f024985bf3ee7
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.