Use /tmp instead of Temp/ in TSNE code

sfu-natlang / lensingwikipedia

Lensing Wikipedia is an interface to visually browse through human history as represented in Wikipedia. This the source code that runs the website:

http://lensingwikipedia.cs.sfu.ca

Other

11 stars 4 forks source link

Use /tmp instead of Temp/ in TSNE code #180

Closed avacariu closed 9 years ago

avacariu commented 9 years ago

The Temp/ folder was created in the Makefile, and was a relative path (restricting what the pwd could be when executing the program). This caused a few issues.

/tmp is a standard place for putting these kinds files and we're usually guaranteed that it will have the correct permissions for writing to it.

Fixes #179

theq629 commented 9 years ago

It might be even better to just use mkdtemp() and not specify the directory at all, because it should use /tmp automatically if that's right for the system.

avacariu commented 9 years ago

That's a good point. I'll get rid of the dir parameter completely.