clstm icon indicating copy to clipboard operation
clstm copied to clipboard

compile on windows?

Open BackT0TheFuture opened this issue 7 years ago • 7 comments

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

BackT0TheFuture avatar Nov 21 '16 19:11 BackT0TheFuture

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 avatar Dec 05 '16 17:12 zuphilip

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.

stweil avatar Dec 05 '16 18:12 stweil

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 avatar Dec 06 '16 10:12 kba

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

kba avatar Dec 06 '16 11:12 kba

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

zuphilip avatar Dec 06 '16 12:12 zuphilip

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.

kba avatar Dec 06 '16 15:12 kba

I see . thanks

BackT0TheFuture avatar Dec 27 '16 15:12 BackT0TheFuture