-
Notifications
You must be signed in to change notification settings - Fork 110
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
Pre-processing of CSV #66
Labels
Comments
CR #83 |
I'd rather not complicate this script by doing additional pre-processing. There are various commands you can run your file through before piping to |
reubano
added a commit
that referenced
this issue
Jun 9, 2022
* features: Bump to version 0.30.0 [NEW] Add option to set last row (fixes #83) [NEW] Add transaction filters (fixes #83) [NEW] Skip initial rows/columns (fixes #66, #67, and #83) [DOC] Update/correct mapping documentation (closes #16) [DOC] Update readme content Fix chunksize default value spacing and update readme [FIX] Make has_header optional (defaults to True) Test foreign currency (closes #70) [NEW] Add date parsing format option (closes #60) [DOC] Clarify how date_fmt is used (close #60) [ENH] Add `dayfirst` option/kwarg (closes #39)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
Great job on the library it's incredibly useful.
Would it make sense to add a pre-processing sequence the same way you have different mappings ?
My bank has incredibly badly formed CSV so I need to pre-process it to have it match the format that csv2ofx can understand. I am assuming this is something that a lot of people have encountered.
The code would look something like :
where
pre_process
would be a function taking a path and returning a StringIO.You could then have pre-processing for a bunch of counterparties that do not know how to output proper CSV.
Does any of this make sense ?
The text was updated successfully, but these errors were encountered: