Page MenuHomeFreeBSD

WIP: Dlib, Torch7, OpenFace
AbandonedPublic

Authored by arrowd on Oct 1 2017, 8:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 16, 4:53 AM
Unknown Object (File)
Tue, Apr 9, 10:53 AM
Unknown Object (File)
Thu, Mar 28, 1:26 PM
Unknown Object (File)
Thu, Mar 28, 6:24 AM
Unknown Object (File)
Tue, Mar 26, 11:43 PM
Unknown Object (File)
Dec 30 2023, 8:32 PM
Unknown Object (File)
Dec 20 2023, 5:51 AM
Unknown Object (File)
Nov 11 2023, 1:44 PM

Details

Reviewers
mat
Group Reviewers
Contributor Reviewers (ports)
Summary

This is WIP diff adding ports for

  • Dlib C++ library
  • Torch7 scientific framework
  • OpenFace python library for face detection

There are numerous TODOs, but it started to get too complex, so I wanted to share my work.

Current problems:

  • Enable support for various Dlib options
  • BLAS support in Dlib doesn't work with default BLAS library (netlib). The problem is subtle, I've notified maintainer about this. At the same time, Torch7 defaults to openblas, so maybe it is better to just use it for Dlib too?
  • There is no luajit support in lua.mk. This made me hard code some paths in the makefiles. Moreover, ports like devel/lua-cwrap can also work with lua5*, and I havem't figured out how to handle this case. The upcoming FLAVORS support might fix this, but I'd rather stick to more old-school handling.

I plan to gradually fix issues and extend torch7-* ports, but any help is welcome.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 11835
Build 12177: arc lint + arc unit

Event Timeline

mat requested changes to this revision.Oct 6 2017, 3:10 PM
mat added inline comments.
devel/lua-cwrap/Makefile
19

Default, remove.

devel/lua-penlight/Makefile
17

PORTNAME should be Penlight, and then you can remove this line.

devel/torch7-sys/Makefile
20

Default, remove.

graphics/py-openface/Makefile
21–24
science/dlib-cpp/Makefile
14–18
science/py-dlib/Makefile
17–21
29–45
science/torch7-main/Makefile
3
PORTNAME=torch7
PKGNAMESUFFIX=-main

And then remove GH_PROJECT.

21

Default, remove.

science/torch7-repl/Makefile
22

Remove

This revision now requires changes to proceed.Oct 6 2017, 3:10 PM
arrowd marked 10 inline comments as done.

Address comments. Support all options in dlib port. Incorporate some upstream fixes.

BLASLAPACK support in both dlib-cpp and py-dlib is currently problematic, because it pulls in
GCC. In dlib-cpp case this leads to crashes in testsuite, while py-dlib simply doesn't compile.

Torch7 ports are also still in bad shape - torch7-repl doesn't seem to find modules installed by other torch7-* packages.
I'm going to address this on next iteration.

Apart from BLASLAPACK option, I think dlib-cpp and py-dlib can be commited.

devel/lua-Penlight/Makefile
19–20 ↗(On Diff #34290)

I do not think hardcoding 5.1 here is a good idea.

ultima added inline comments.
devel/lua-Penlight/pkg-plist
1 ↗(On Diff #34290)

Also not a good idea to hard code the lua version in plist, when the default version changes or set to a different value all (or at least most) of these ports will break.

Bump. Can dlib-cpp and py-dlib be committed?

Torch isn't actively developed anymore.