symforce icon indicating copy to clipboard operation
symforce copied to clipboard

Should add an option to find external metis

Open chao-qu-skydio opened this issue 3 years ago • 3 comments

Either this or we need to add its license in the repo for creating conda package.

#212

chao-qu-skydio avatar Nov 28 '22 19:11 chao-qu-skydio

Can we clarify exactly what this means? Do we just need to add an initial attempt to find_package(metis <some min version?> QUIET) here like we do for other dependencies? And then have some ability to build wheels (or just a conda package) that dynamically link against libmetis.so? I believe we currently statically link against libmetis in wheel builds.

Or do we just need to include a notice in LICENSE that binary distributions of symforce contain METIS with a reference to its license?

I'd be fine with either of these, we should probably have an option to do the first one (at least for cmake builds if not all the piping for wheels) either way?

aaron-skydio avatar Nov 30 '22 03:11 aaron-skydio

Yeah, I meant adding a find_package(METIS). We could use (or modify) the cmake file from here https://github.com/ceres-solver/ceres-solver/blob/master/cmake/FindMETIS.cmake

chao-qu-skydio avatar Nov 30 '22 04:11 chao-qu-skydio

SGTM, feel free to go for it

aaron-skydio avatar Nov 30 '22 04:11 aaron-skydio