Added dtype to images.map

[boazmohar]

fixes #284
Only adds the functionality for images object not for series.

[boazmohar]

Added dtype to series as well.
Something to consider in series.map()
There is currently no way to pass a shape of the resulting index without passing an index. It is an edge case, and I don't know if you want this to be implimented.
@freeman-lab What do you think?

[jwittenbach]

@boazmohar this is looking great. Any chance you could add the same functionality to Blocks.map + a corresponding test? If we can get that in, then I'd be happy to merge this in.

[boazmohar]

Sure I will do it later today.

Thanks,
Boaz On Mon, May 09, 2016 at 1:09 PM, Jason Wittenbach < notifications@github.com [notifications@github.com] > wrote:
@boazmohar [https://github.com/boazmohar] this is looking great. Any chance you could add the same functionality to Blocks.map + a corresponding test? If we can get that in, then I'd be happy to merge this in.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub [https://github.com//pull/285#issuecomment-217926240]

[boazmohar]

@jwittenbach looking over the blocks.py code, there is a dtype passed there:
blocks.py@map()

I haven't seen a test for it, do you want me to add one?

[jwittenbach]

@boazmohar ah, I should have looked more closely. Yeah, if you want to add a test, that would be perfect!

[boazmohar]

@jwittenbach Done, could you please have a look, I want Aaron to use this fix as well.
Thanks!

[jwittenbach]

@boazmohar awesome. Looks like there are some merge conflicts. This is probably because some other changes have been made in the meantime. If you do a git pull upstream from this branch on your local machine, then you it will try to grab those changes. Any files where those changes clash with yours will show up when you do git status. Inside those files it will show you the conflict, and you just need to manually resolve it. Then do another comit and push.

[boazmohar]

@jwittenbach sorry about the mess, should work now.

[jwittenbach]

Looks good to me. Thanks @boazmohar!