Skip to content
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

compile on windows? #113

Open
BackT0TheFuture opened this issue Nov 21, 2016 · 7 comments
Open

compile on windows? #113

BackT0TheFuture opened this issue Nov 21, 2016 · 7 comments
Labels

Comments

@BackT0TheFuture
Copy link

BackT0TheFuture commented Nov 21, 2016

hi,there
CLSTM is great , do you guys have some plan for windows support? cmake or MinGW
thx!

@BackT0TheFuture BackT0TheFuture changed the title windows support ! compile on windows ! Dec 3, 2016
@zuphilip
Copy link
Collaborator

zuphilip commented Dec 5, 2016

This is IMO quite unrealistic at the moment, but you can try to work with docker. There is a Dockerfile of the project. (@kba is this still up-to-date?)

@zuphilip zuphilip changed the title compile on windows ! compile on windows? Dec 5, 2016
@stweil
Copy link
Contributor

stweil commented Dec 5, 2016

The current code uses functions like glob which are not available with Minw-w64. That code needs modifications to make it more portable if you want to support Windows hosts.

And of course you also need Eigen built for Windows.

@kba
Copy link
Collaborator

kba commented Dec 6, 2016

With this project essentially in maintenance mode (@tmbdev is developing a new LSTM/OCR project) I do not think a native windows port is worth the porting/testing effort. But if someone takes the initiative, I will support it.

As @zuphilip mentions, docker is an option, though not the Dockerfile bundled in the repository, which was for testing CUDA/GPU stuff and is outdated. I have an up-to-date Dockerfile for clstm somewhere, I'll post it when I find it. It's essentially executing the installation steps in the README.

@kba
Copy link
Collaborator

kba commented Dec 6, 2016

Here's the Dockerfile:

FROM ubuntu:16.04
ENV DEBIAN_FRONTEND noninteractive

WORKDIR /tmp
RUN apt-get -y update
RUN apt-get -y install git scons libprotobuf-dev libprotobuf9v5 protobuf-compiler g++ libpng12-dev
RUN git clone --depth 1 --single-branch --branch 3.3-rc1 \
	  "https://github.com/RLovelett/eigen" /usr/local/include/eigen3
RUN git clone --depth 1 "https://github.com/tmbdev/clstm"

WORKDIR /tmp/clstm
RUN scons && scons all

@zuphilip
Copy link
Collaborator

zuphilip commented Dec 6, 2016

Shouldn't we add this Dockerfile as well to the repo and maybe rename the current one?

@kba
Copy link
Collaborator

kba commented Dec 6, 2016

I have them both ("regular" and updated CUDA) in a separate folders. The CUDA deps pull in lots of GUI packages (>4GB image) and there's little benefit, so it's more for curiosity. I'll send a PR.

@BackT0TheFuture
Copy link
Author

I see . thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants