-
Notifications
You must be signed in to change notification settings - Fork 55
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
feat(engine): added main engine mvp #7
Conversation
1ee36c6
to
87c3d41
Compare
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.
Looking good, happy to see it going through the notebook test with apparently no trouble!
There are probably still some compatibility issues depending on the provided arguments.
Integration can probably still go a long way (mitigating duplicated resources/tasks) but we are getting there 🙌🏼
1ddf57a
to
c1277cb
Compare
Implements the core DataQuality engine that dispatches and stores results from the multiple individual engines.
Features:
Closes #6