Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

[Windows] DLL Path and Name #314

Closed
tqchen opened this issue Oct 18, 2015 · 15 comments
Closed

[Windows] DLL Path and Name #314

tqchen opened this issue Oct 18, 2015 · 15 comments

Comments

@tqchen
Copy link
Member

tqchen commented Oct 18, 2015

@hjk41 Is it possible to make the windows dll libmxnet.dll instead of mxnet.dll? Since the later will be used by R libraries.

@tqchen
Copy link
Member Author

tqchen commented Oct 18, 2015

@yajiedesign

It is also great if we can point the build result path to lib/libmxnet.dll so it is consistent with linux/osx

@hjk41
Copy link
Contributor

hjk41 commented Oct 19, 2015

ok

On Sun, Oct 18, 2015 at 11:59 AM, Tianqi Chen notifications@github.com
wrote:

@yajiedesign https://github.com/yajiedesign

It is also great if we can point the build result path to lib/libmxnet.dll
so it is consistent with linux/osx


Reply to this email directly or view it on GitHub
#314 (comment).

HONG Chuntao
System Research Group
Microsoft Research Asia

@thirdwing
Copy link
Contributor

@hjk41 Can you share the dll with me? Or can you help test the R package on Windows? I can tell you the steps.

@hjk41
Copy link
Contributor

hjk41 commented Oct 19, 2015

https://github.com/dmlc/mxnet/releases/download/untagged-3801d7b98bea4a33b21c/mxnet_0.9_win64_cpu_only.zip

This is a draft and is very large: ~600MB. I am now working to shrink the
size. Please use it for now and I will post the refined package soon.

On Mon, Oct 19, 2015 at 11:48 AM, Qiang Kou (KK) notifications@github.com
wrote:

@hjk41 https://github.com/hjk41 Can you share the dll with me? Or can
you help test the R package on Windows? I can tell you the steps.


Reply to this email directly or view it on GitHub
#314 (comment).

HONG Chuntao
System Research Group
Microsoft Research Asia

@hjk41
Copy link
Contributor

hjk41 commented Oct 19, 2015

I have just compiled a version with statically linked opencv:

https://github.com/dmlc/mxnet/releases/download/untagged-175e0487d0d7aaff577b/mxnet_0.9_win64_cpu_only.7z

Please use this one and see if it works with R.

On Mon, Oct 19, 2015 at 12:34 PM, Chuntao HONG chuntao.hong@gmail.com
wrote:

https://github.com/dmlc/mxnet/releases/download/untagged-3801d7b98bea4a33b21c/mxnet_0.9_win64_cpu_only.zip

This is a draft and is very large: ~600MB. I am now working to shrink the
size. Please use it for now and I will post the refined package soon.

On Mon, Oct 19, 2015 at 11:48 AM, Qiang Kou (KK) <notifications@github.com

wrote:

@hjk41 https://github.com/hjk41 Can you share the dll with me? Or can
you help test the R package on Windows? I can tell you the steps.


Reply to this email directly or view it on GitHub
#314 (comment).

HONG Chuntao
System Research Group
Microsoft Research Asia

HONG Chuntao
System Research Group
Microsoft Research Asia

@tqchen
Copy link
Member Author

tqchen commented Oct 19, 2015

Thanks @hjk41 The file size is great. Do you think if you can give a script to "INSTALL" the library? I.e. put it in some location and have a script to setup Path automatically

@tqchen
Copy link
Member Author

tqchen commented Oct 19, 2015

As far as I understand, mxnet_shell.cmd only works for this command line, but did not setup env for the system

@hjk41
Copy link
Contributor

hjk41 commented Oct 19, 2015

ok... I can make a installer for this.

On Mon, Oct 19, 2015 at 2:46 PM, Tianqi Chen notifications@github.com
wrote:

As far as I understand, mxnet_shell.cmd only works for this command line,
but did not setup env for the system


Reply to this email directly or view it on GitHub
#314 (comment).

HONG Chuntao
System Research Group
Microsoft Research Asia

@tqchen
Copy link
Member Author

tqchen commented Oct 19, 2015

I think we like to have the zip version, so advanced user can put things wherever they like and setup path manually. And a additional setup script in the zip to help user to install things when they donnot know what to do

@thirdwing
Copy link
Contributor

@hjk41 Hi, just a small problem. R on Windows is compiled using sjlj and you are using seh.

Besides, it seems to depend on VCOMP120.dll. (I have really little knowledge about Windows, so I don't know what it is.)

Can you give some instructions on how to compile it on Windows using MinGW-w64?

@tqchen
Copy link
Member Author

tqchen commented Oct 19, 2015

@thirdwing The dll might be VS 2013 runtime. The exception handling mechanism should be fine, as we catch most exceptions in the libmxnet.dll and return a 0 in the C API. So exception would not cross the dll boundary

@thirdwing
Copy link
Contributor

After switching to sjlj, we don't need to include libgcc_seh.dll.

@hjk41
Copy link
Contributor

hjk41 commented Oct 20, 2015

libgcc_seh.dll is the dependency introduced by openblas.

On Tue, Oct 20, 2015 at 6:42 AM, Qiang Kou (KK) notifications@github.com
wrote:

After switching to sjlj, we don't need to include libgcc_seh.dll.


Reply to this email directly or view it on GitHub
#314 (comment).

HONG Chuntao
System Research Group
Microsoft Research Asia

@thirdwing
Copy link
Contributor

So you are using precompiled openblas? It will be better if you can compile it using sjlj. But it doesn't matter, I think.

@tqchen tqchen closed this as completed Oct 20, 2015
@tqchen
Copy link
Member Author

tqchen commented Oct 20, 2015

move to #287

eric-haibin-lin pushed a commit to eric-haibin-lin/mxnet that referenced this issue Dec 2, 2017
* [PARAMETER] Print bool value as 0/1 to make parsing easier

* add defult value to type name
iblislin added a commit to iblislin/incubator-mxnet that referenced this issue Mar 18, 2018
* sym: broadcast unfusion for `add`

* sym: broadcast unfusion for `minus`

* sym: broadcast unfusion for `multiplication`

* sym: broadcast unfusion for `div`

* sym: broadcast unfusion for `power`

* sym: broadcast unfusion for `power` with irrational
iblislin added a commit to iblislin/incubator-mxnet that referenced this issue Mar 18, 2018
* prepare for releasing

* build: fix windows build

* update NEWS for apache#314

* update NEWS for apache#312

* update NEWS for apache#276
eric-haibin-lin pushed a commit to eric-haibin-lin/mxnet that referenced this issue Apr 4, 2018
* Keras keeps renaming properties. Update frontend for keras 2.1.3 compatibility

* Add error message when inbound_nodes is not found
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants