Improving MethodError message for anonymous functions#57319
Closed
Bumblebee00 wants to merge 3 commits intoJuliaLang:masterfrom
Closed
Improving MethodError message for anonymous functions#57319Bumblebee00 wants to merge 3 commits intoJuliaLang:masterfrom
Bumblebee00 wants to merge 3 commits intoJuliaLang:masterfrom
Conversation
nsajko
reviewed
Feb 9, 2025
| catch ex | ||
| @error "Error showing method candidates, aborted" exception=ex,catch_backtrace() | ||
| end | ||
| # if the function has only on method, print a Tip about the number of arguments to be used |
Member
There was a problem hiding this comment.
Suggested change
| # if the function has only on method, print a Tip about the number of arguments to be used | |
| # if the function has only one method, print a tip about the number of arguments to be used |
Member
|
I think these are good changes, but IMO it might be better as two separate pull requests, seeing as it's really two unrelated changes:
|
This was referenced Feb 10, 2025
Contributor
Author
|
that is a sensible opinion, i created two different pull requests for the two features |
inkydragon
pushed a commit
that referenced
this pull request
Feb 26, 2025
If a MethodError arises on a anonyomous function, the words "anonymous function" are printed in the error like so: ```julia g=(x,y)->x+y g(1,2,3) ``` ``` ERROR: MethodError: no method of the anonymous function var"#5#6" matching (::var"#5#6")(::Int64, ::Int64, ::Int64) The function `#5` exists, but no method is defined for this combination of argument types. Closest candidates are: (::var"#5#6")(::Any, ::Any) @ Main REPL[4]:1 ``` See the [original pull request](#57319) and [issue](#56325) #56325
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This pull request aims to resolve issue #56325
In this example, you get a MethodError for your anonymous method
(::var"#3#4"):This can be confusing for beginner julia programmers, because in the case of a do block, it might even be non-obvious that you wrote a function. I changed the error message adding the words "anonymous function" and adding a tip for the number of arguments:
Of cousre if the function is not anonymous the error will not say "anonymous function" and if has more than one method will not display the tip:
I have done this by modifying the files
base/errorshow.jlandtest/errorshow.jl