-
Notifications
You must be signed in to change notification settings - Fork 6
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
fix: add ignore_error executor field #176
Comments
Thanks for reporting. This issue is due to Funnel, but not proTES, having adopted TES v1.1.0. The library that throws this error is py-tes. A few points:
Do you want to take care of these issues, @athith-g? That is:
|
@uniqueg For sure I'll update the proTES models and open up that issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
TES v1.1 defines an
ignore_error
field as a property of an executor (defined here). Currently, the TesExecutor model does not have this attribute. As a result, the following error occurs when a response with theignore_error
is provided:TypeError: Executor.__init__() got an unexpected keyword argument 'ignore_error'
Full traceback:

Executor causing error:

Software
Using proTES with Funnel
The text was updated successfully, but these errors were encountered: