Skip to content

Conversation

@9prady9
Copy link
Member

@9prady9 9prady9 commented Dec 17, 2015

Fixes #832

umar456 and others added 30 commits August 9, 2015 01:18
This fixed `Assign.LinearAssignSeq` unit test in assign unit tests.
This also fixed assign unit test: `ArrayAssign.CPP_ASSIGN_VECTOR_2D`
@9prady9
Copy link
Member Author

9prady9 commented Dec 21, 2015

build arrayfire ci

@shehzan10
Copy link
Member

build arrayfire windows ci

@pavanky
Copy link
Member

pavanky commented Dec 28, 2015

@9prady9 Can you make sure cmake detects if the threads submodule is initialized or not (similar to what we are doing with assets).

Right now it fails only when trying to compile

@pavanky
Copy link
Member

pavanky commented Dec 28, 2015

Code looks good. Things to change require the following:

  • CMake checks if threads submodule has been initialized
  • Add getQueue().sync() after each getQueue.enqueue() in debug mode.
  • Add the ability to make the algorithm synchronous at runtime based on environment variable.

@pavanky
Copy link
Member

pavanky commented Dec 28, 2015

@umar456 What do you think about the last point ?

@shehzan10
Copy link
Member

It would be nice to have a compile time option

@9prady9
Copy link
Member Author

9prady9 commented Dec 28, 2015

build arrayfire ci

@umar456
Copy link
Member

umar456 commented Dec 29, 2015

What would be the benefit of having both? Couldn't you just export the variable and make it persistent?

@pavanky
Copy link
Member

pavanky commented Dec 29, 2015

@umar456 try setting and unsetting the variable on windows. It is not pretty.

@pavanky
Copy link
Member

pavanky commented Dec 30, 2015

@9prady9 Can you pull from devel and resolve merge conflicts? You will need to fix transform.cpp in CPU backend.

@9prady9
Copy link
Member Author

9prady9 commented Dec 30, 2015

@pavanky I will start on this after 3.2.2 -> devel PR. I believe that is the last hot fixes before v3.3.0 release, is that right ?

@pavanky
Copy link
Member

pavanky commented Dec 30, 2015

@9prady9 Sure.

@pavanky pavanky added this to the 3.3.0 milestone Dec 30, 2015
umar456 and others added 2 commits December 30, 2015 10:41
This reverts commit 95d9346.

* Not necessary with the new queue class (see queue.hpp)
* Macros bad
@shehzan10
Copy link
Member

build arrayfire ci

pavanky added a commit that referenced this pull request Dec 31, 2015
Changes to make CPU backend functions asynchronous calls
@pavanky pavanky merged commit 861808e into arrayfire:devel Dec 31, 2015
@umar456 umar456 deleted the async branch May 17, 2017 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants