-
Notifications
You must be signed in to change notification settings - Fork 301
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
Include validation check for single table auto_assign_transformers #2021
Conversation
Yes it does, I added a test to multitable to confirm it as well. Multitable synthesizer handles |
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.
Looks good!
I just have one question
@@ -32,7 +32,7 @@ def get_metadata(self, add_sequence_key=True, add_sequence_index=False): | |||
|
|||
def get_data(self): | |||
data = pd.DataFrame({ | |||
'time': [1, 2, 3], |
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.
I actually use datetimes as now auto_assign_transformer
will validate this data which would error out. It should be date-time anyway
resolves #1509
CU-85ztjjzrw
Include better messsaging for auto_assign_transformers by checking metadata and data before going through assigning the transformers.