search

gonum / blas

A BLAS implementation for Go [DEPRECATED]
172 stars 16 forks source link

cgo: note that BLAS is underspecified #167

Closed kortschak closed 8 years ago

kortschak commented 8 years ago

@btracey Please take a look.

Fixes #118.

btracey commented 8 years ago

Lgtm. We probably want a similar disclaimer in native saying that blas didn't specify man and that the behavior of native with nan may change

kortschak commented 8 years ago

We probably want a similar disclaimer in native saying that blas didn't specify man and that the behavior of native with nan may change

How would we change it? and why?

btracey commented 8 years ago

Efficiency reasons. I don't have any direct plans, but it seems like ignoring NaNs to various degrees can be notably faster (looking at the openblas thread)

kortschak commented 8 years ago

We can probably leave the note for now since we say that the blas implementations follow the BLAS standard.