Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

[WIP] Initial implementation of DateTimeSplitter #403

Closed
wants to merge 9 commits into from

Conversation

pieths
Copy link
Collaborator

@pieths pieths commented Dec 30, 2019

There is a bug in DateTimeTransformer in ML.Net related to country selection. This is expected to be fixed in the next version of ML.Net (probably 1.5.0-preview2). Until then, the unit test and example which use this feature will not pass.

@@ -10,6 +10,7 @@ lib_lightgbm.dylib
libtensorflow.dylib
libonnxruntime.dylib
libtensorflow_framework.1.dylib
Featurizers.dll
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not .so file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought that was weird too. This is what was in the original file in the aml branch and there are only *.dylib or *.dll references in that file.

@pieths
Copy link
Collaborator Author

pieths commented Jan 24, 2020

This has been superseded by #415.

@pieths pieths closed this Jan 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants