-
Notifications
You must be signed in to change notification settings - Fork 62
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: Onboard New Fec dataset #486
Conversation
Gowtham, there are multiple issues with the transform_csv python file: (1) Please remove ALL blank lines between code. There should be no inter-function blank lines. |
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.
As specified in my comment
datasets/fec/pipelines/_images/run_csv_transform_kub/csv_transform.py
Outdated
Show resolved
Hide resolved
datasets/fec/pipelines/_images/run_csv_transform_kub/csv_transform.py
Outdated
Show resolved
Hide resolved
datasets/fec/pipelines/_images/run_csv_transform_kub/csv_transform.py
Outdated
Show resolved
Hide resolved
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.
Thanks Gowtham.
@gkodukula This is now merged and deployed to production. Please ensure execution cycle completes successfully. |
Description
Dataset: FEC
Checklist
Use the sections below based on what's applicable to your PR and delete the rest:
Feature
README
accordinglyData Onboarding
datasets/fec
and nothing outside of that directory