Closed 9il closed 9 years ago
ping @kyllingstad
Hi, Sorry for the silence. I've been very busy lately, and haven't had time to look at this yet. I'll try to get to it this weekend!
Thanks!
BLAS is not actually used directly in SciD, only LAPACK. The bindings in scid.bindings.blas.*
were only included for completeness when I copied the LAPACK bindings from Bill Baxter's MultiArray library (for D1) back in the days. I'd rather not add a dependency on the cblas
package when it's not necessary.
The change from built-in complex types to std.complex.Complex
, though, is great.
I'll add some more comments inline.
My suggestions:
dub.json
deprecated
statements, but maybe you should add an URL to the cblas
GitHub or code.dlang.org
page to the messages?scid.bindings.lapack.lapack
and keep the alias
declarations there (directly declaring f_int
to be int
).Fixed
Thanks!
Thank you for great library !
I'll bump to v0.2.0 as soon as I've verified that the new configuration file for DDocs.org works. (I had to turn off Markdown, because it messed up some of the documentation.
OK. If MD turned off you may want to remove last commit https://github.com/9il/scid/commit/62f69b2d31498431d1843a58d9c601ba2fd84935 in this PR.
Oh, right. I didn't notice that. Fixed now.
cblas
package is used instead of