-
Notifications
You must be signed in to change notification settings - Fork 0
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
deal with the malformed CSV text from IEA-EV-dataEVsalesCarsProjection-APS.csv
#46
Conversation
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 think that this is a fine solution, but I think that RMI-PACTA/workflow.scenario.preparation#48 is a simpler option that we could consider.
The goal is to keep file i/o in workflows, and data munging in R pkgs, so this more accurately follows that strategy. |
@AlexAxthelm @cjyetman am I the only person that feels like this is an absurdly huge hoop to jump through just to reformat a file that was misformatted due to our manual error to begin with...?! lol like just update the file!? |
no... that was also my preferred solution |
If you both feel strongly that editing the file is the better call here, then we can do that. I'll make the changes and upload to AFS. |
Thanks @AlexAxthelm Closing this PR then. |
This causes a breaking change in workflow.scenario.preparation that will be fixed by RMI-PACTA/workflow.scenario.preparation#49