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

Use f-ordered result array and a more comparable gemm invocation for ND4J #46

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@treo

treo commented Jun 26, 2018

No description provided.

@blueberry

This comment has been minimized.

Member

blueberry commented Jun 27, 2018

Thank you for this update. I'll add gemm in addition to mmul, instead of replacing it. Neanderthal's mm does much more than gemm. Being polymorphic not only regarding the first matrix, but also the second, it does lots of things. gemm is, on the other hand, a static method that straightforwardly does gemm. Nevermind, Neanderthal still comes out ahead, so why not accept gemm? :)

I'll add your changes as addition to the existing code outside this PR. When can I expect the release of the beta2 of ND4J, so I can update the project to use ND4J for the recent bugfixes related to this? Snapshot does not work for me, and will probably not work for other users, since it picks up OpenBLAS instead of MKL.

@blueberry blueberry closed this Jun 27, 2018

@treo

This comment has been minimized.

treo commented Jun 27, 2018

With snapshots, and probably with any upcoming versions you've got to explicitly tell ND4J (or rather JavaCPP which can pickup any BLAS Library via the Openblas Bindings) what you'd like to use:
https://github.com/bytedeco/javacpp-presets/tree/master/openblas

For my own tests I used -Dorg.bytedeco.javacpp.openblas.load=mkl_rt.

Concerning the next release, all that I'm aware of is that it isn't going to be very soon.

@blueberry

This comment has been minimized.

Member

blueberry commented Jun 27, 2018

Thanks for the info. I'll use snapshots.

@blueberry

This comment has been minimized.

Member

blueberry commented Jun 27, 2018

Do you have any maven repository with snapshots that I can use? If I switch to the snapshot of ND4J, that means that whoever would like to try these benchmarks would have to build the whole DL4J/ND4J themselves, which hardly anyone is willing to do...

@treo

This comment has been minimized.

treo commented Jun 27, 2018

See https://github.com/treo/benchmarking_nd4j/blob/master/pom.xml. I'm using the snapshots repository there.

@blueberry

This comment has been minimized.

Member

blueberry commented Jun 27, 2018

Unfortunately, leiningen complains about the security certificate of sonatype's snapshots repository and refuses to download from it...

I'll leave 1.0.0-beta for now in the repo, but hope someone will resolve this.

@treo

This comment has been minimized.

treo commented Jun 27, 2018

Huh, that weird. As far as I can tell leiningen should be using java's trust store to check them, and that certificate should be valid with any up to date java jre, i.e. java 8 update version >= 31.

@blueberry

This comment has been minimized.

Member

blueberry commented Jun 27, 2018

Yes, but it complains and the jar is not downloaded. Anyway, I've updated the project.clj, and commented out the offending part, so anyone can try, and may have more luck with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment