-
Notifications
You must be signed in to change notification settings - Fork 31
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
UDF improvements #2132
UDF improvements #2132
Conversation
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This looks like an attempt to have a custom python-like parser. Using the
running the above gives:
|
Thanks for pointing that out. I've rewritten this using ast parsing, and also have included support for function calls in expressions, which can be present in MDF models. |
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This pull request introduces 1 alert and fixes 1 when merging 89c7a7f into e20a163 - view on LGTM.com new alerts:
fixed alerts:
|
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
previously required them in dictionary form as the params argument
This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):
See CI logs for the full diff. |
This pull request introduces 1 alert and fixes 1 when merging 773e11d into 84d0aa3 - view on LGTM.com new alerts:
fixed alerts:
|
stateful_parameter
argument