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

__array_priority__ regression #81

Closed
ZacDiggum opened this issue May 11, 2016 · 6 comments
Closed

__array_priority__ regression #81

ZacDiggum opened this issue May 11, 2016 · 6 comments

Comments

@ZacDiggum
Copy link
Contributor

ZacDiggum commented May 11, 2016

Hi,
I just installed the third v3.3 bugfix release. The problem with adding / multiplying numpy scalars and af arrays somehow came back. See the discussion with @FilipeMaia, issue #69.
Try something like:

a = af.randu(5,5)
b = np.sin(1)*a

b will be a numpy array. Back then, Filipe changed the __array_priority__ attribute.

@ZacDiggum
Copy link
Contributor Author

ZacDiggum commented May 11, 2016

I just noticed this only happens with this version which is master, I guess. Devel works fine.

@pavanky
Copy link
Member

pavanky commented May 11, 2016

@ZacDiggum I don't think I ever merged that into master.. That PR was sent to devel.

@ZacDiggum
Copy link
Contributor Author

:-) Are you going to merge it?

@pavanky
Copy link
Member

pavanky commented May 11, 2016

@ZacDiggum I am going to do it. I have come across a couple of other minor issues as well that need to be resolved.

@pavanky pavanky reopened this May 11, 2016
@pavanky
Copy link
Member

pavanky commented May 11, 2016

I'll close it when the issue is resolved. Makes sure I am keeping track of things :)

@pavanky
Copy link
Member

pavanky commented May 16, 2016

Ok this is now fixed :) sorry about the delay. I hit a bug when testing with unified backend and it ended up being an upstream bug in arrayfire itself :)

@pavanky pavanky closed this as completed May 16, 2016
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

No branches or pull requests

2 participants