search

tpapp / lla

Lisp Linear Algebra
Boost Software License 1.0
90 stars 16 forks source link

latest cl-num-utils with-indexing and with-indexing* breaks lla #4

Closed moshelooks closed 13 years ago

moshelooks commented 13 years ago

Hi Tamas,

The latest changes to cl-num-utils with-indexing and with-indexing* break the lla code that calls them. I've put a fix at

https://github.com/moshelooks/lla

what do you think?

Thanks, Moshe

P.S. This is a really nice library - great, clean design! P.P.S. Any chance you could refactor to eliminate the dependency on the deprecated tpapp-utils?

tpapp commented 13 years ago

Hi Moshe,

LLA is currently undergoing a reorganization -- it will use the latest LAPACKE C interface, which will allow the use of native, row-major CL arrays as matrices, making it even better integrated into CL. I am currently working on this, the new version will be released in a few weeks. In the meantime, LLA remains broken, I am sorry about this. I didn't think anyone else was using the library.

Tamas

On Wed, 13 Apr 2011, moshelooks wrote:

Hi Tamas,

The latest changes to cl-num-utils with-indexing and with-indexing* break the lla code that calls them. I've put a fix at

https://github.com/moshelooks/lla

what do you think?

Thanks, Moshe

P.S. This is a really nice library - great, clean design! P.S.S. Any chance you could refactor to eliminate the dependency on the deprecated tpapp-utils?

moshelooks commented 13 years ago

Tamas,

LLA is currently undergoing a reorganization -- it will use the latest LAPACKE C interface, which will allow the use of native, row-major CL arrays as matrices, making it even better integrated into CL.  I am currently working on this, the new version will be released in a few weeks.  In the meantime, LLA remains broken, I am sorry about this.  I didn't think anyone else was using the library.

No problem, a few weeks wait for an even better LLA is fine. Thank you for the prompt and helpful response!

  • Moshe
tpapp commented 13 years ago

Hi Moshe,

It turns out that the wait was a bit more than a few weeks, but now I have preliminary version that works, and I merged it to the master branch. Things should work now, but remember to get the latest dependencies (cl-num-utils, let-plus).

Best,

Tamas

moshelooks commented 13 years ago

Thanks, Tamas! This looks really nice, I'm looking forward to exploring all of the improvements.

On Wed, Jul 20, 2011 at 2:30 AM, tpapp reply@reply.github.com wrote:

Hi Moshe,

It turns out that the wait was a bit more than a few weeks, but now I have preliminary version that works, and I merged it to the master branch.  Things should work now, but remember to get the latest dependencies (cl-num-utils, let-plus).

Best,

Tamas

Reply to this email directly or view it on GitHub: https://github.com/tpapp/lla/issues/4#issuecomment-1614079