Skip to content

allow Reference as callable in map/apply#1222

Merged
minrk merged 1 commit into
ipython:masterfrom
minrk:mapreference
Jan 16, 2012
Merged

allow Reference as callable in map/apply#1222
minrk merged 1 commit into
ipython:masterfrom
minrk:mapreference

Conversation

@minrk
Copy link
Copy Markdown
Member

@minrk minrk commented Jan 2, 2012

Assumptions were made that the first argument was a callable/function with
a __name__ attribute. These assumptions were the only barrier to using
References, and have been removed.

Associated tests included.

Assumptions were made that the first argument was a callable/function with
a __name__ attribute.  These assumptions were the only barrier to using
References, and have been removed.

Associated tests included.
@ellisonbg
Copy link
Copy Markdown
Member

This is an important optimization of the parallel map functionality. The implementation is clean and it is ready for merging.

minrk added a commit that referenced this pull request Jan 16, 2012
allow Reference as callable in map/apply

This allows behaviors like:

    rf = parallel.Reference('f')
    ar = view.map(rf, *sequences)

where the function 'f' is defined on the engines, and can even be different.
@minrk minrk merged commit 5608bfe into ipython:master Jan 16, 2012
minrk added a commit that referenced this pull request Jan 18, 2012
allow Reference as callable in map/apply

This allows behaviors like:

    rf = parallel.Reference('f')
    ar = view.map(rf, *sequences)

where the function 'f' is defined on the engines, and can even be different.
mattvonrocketstein pushed a commit to mattvonrocketstein/ipython that referenced this pull request Nov 3, 2014
allow Reference as callable in map/apply

This allows behaviors like:

    rf = parallel.Reference('f')
    ar = view.map(rf, *sequences)

where the function 'f' is defined on the engines, and can even be different.
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