-
Notifications
You must be signed in to change notification settings - Fork 4
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
add support for random UDF for x-field-pattern fields #467
Conversation
|
||
# TODO; temporary hack to add auto hierarchy field at the end, remove later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make sure we cover following scenario before removing TODO
- Test should not regress, expecting you have some test.
- I couldn't figure out proper fix of this todo. Could you please pointing out that change.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Earlier the fix was to add new auto parameters to end by default forcefully , now the solution is that the features are added according to their position in the features list
Introducing New random feature for x-field-pattern
Random for Ipv4 Pattern
Random for mac pattern
Random for integer pattern
sample python code to use random
sample go code to use random