Skip to content
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

Better error reporting on overload #19

Closed
Thrameos opened this issue Jul 13, 2017 · 1 comment
Closed

Better error reporting on overload #19

Thrameos opened this issue Jul 13, 2017 · 1 comment
Assignees

Comments

@Thrameos
Copy link
Owner

When executing a complicated command while using a builder pattern such as

sg=sgb.add(bkg).includeWeightedSNR(sg1,10.0).miscalibrate(0, 0.01).create()

it is not clear which overloaded method is causing an error.

RuntimeError: No matching overloads found. at native\common\jp_method.cpp:117

At bare minimum the name of the method should appear in the error message. Although the native line number is useful for the developer it is likely meaningless for the user.

@Thrameos Thrameos self-assigned this Jul 13, 2017
@Thrameos
Copy link
Owner Author

Thrameos commented Jul 3, 2018

Fixed in current master and devel.

@Thrameos Thrameos closed this as completed Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant