Skip to content
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

Add append as a keyword argument to save #64

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pearlzli
Copy link

@pearlzli pearlzli commented Sep 28, 2019

This currently fails the savestreaming test in a way that I can't make any sense of: I get a MethodError: no method matching -(::String, ::String) thrown by FileIO.savestreaming. Does anyone have any ideas?
Fixed, and now all tests pass (9/29)

Also, I've chosen not to try to implement appending to a gzipped CSV.

Thanks in advance for your help! I'm pretty new to how things work in this package...

src/csv_writer.jl Outdated Show resolved Hide resolved
@pearlzli
Copy link
Author

@davidanthoff That was exactly it, thank you! I feel a little silly now...

@pearlzli pearlzli changed the title WIP: Add append as a keyword argument to save Add append as a keyword argument to save Sep 29, 2019
@pearlzli
Copy link
Author

pearlzli commented May 5, 2020

Hi @davidanthoff - have you had the chance to look at this at all? Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants