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.

Sign up for GitHub

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

Jump to bottom

Wrong default argument of DirectView.clear #4103

Closed
WKarel opened this issue Aug 24, 2013 · 1 comment · Fixed by #4106
Closed

Wrong default argument of DirectView.clear #4103

WKarel opened this issue Aug 24, 2013 · 1 comment · Fixed by #4106
Labels
bug parallel
Milestone
2.0

Comments

@WKarel
Copy link

WKarel commented Aug 24, 2013

DirectView.clear
in
parallel/client/view.py
line 800
seems to have the wrong default-argument for block:
def clear(self, targets=None, block=False)

It should probably be None instead of False.
minrk added a commit to minrk/ipython that referenced this issue Aug 24, 2013
@minrk
fix a couple of default block values
3670ae7 
should be None

closes ipython#4103

should be backported to 1.1
@minrk minrk mentioned this issue Aug 24, 2013
Merged
@minrk
Copy link
Member

minrk commented Aug 24, 2013

should be fixed by #4106

takluyver added a commit that referenced this issue Aug 27, 2013
@takluyver
Backport PR #4106: fix a couple of default block values
3d2f9d3 
should be None

closes #4103

should be backported to 1.1
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this issue Nov 3, 2014
@minrk
fix a couple of default block values
a4bd802 
should be None

closes ipython#4103

should be backported to 1.1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug parallel
Projects
None yet
Milestone
2.0
Development

Successfully merging a pull request may close this issue.

fix a couple of default block values minrk/ipython
2 participants
@minrk @WKarel