Skip to content

progress callback doesn't work #16

Closed
coryvirok opened this Issue Dec 3, 2012 · 3 comments

2 participants

@coryvirok
fp = ProgressCallingFile(fp, progress)
...

TypeError: __init__() takes exactly 4 arguments (3 given)
@lericson
Owner

Correct, it should be fixed. The signature is actually ProgressCallingFile(fp, size, progress). Where did you get this? The documentation?

@coryvirok

Haven't tried it since I filed the issue, but I got the usage from here: https://github.com/lericson/simples3/blob/master/simples3/streaming.py#L64

@lericson
Owner
@lericson lericson added a commit that referenced this issue Dec 24, 2012
@lericson Add test coverage to streaming
Prevents future issues like #16
d88bfec
@lericson lericson closed this in 7c01f0b Dec 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.