isaacarvestad/edc

Expander decomposition of graphs.

edc's Issues

Build of edc-cut

Dear Isaac,

Thanks for this amazing repo. I have a question about the build of edc-cut.

When building edc-cut with bazel, I tried
bazel build -c opt //main:edc-cut

and it reports the following error:

INFO: Build options --compilation_mode, --cxxopt, and --linkopt have changed, discarding analysis cache.
INFO: Analyzed target //main:edc-cut (0 packages loaded, 372 targets configured).
INFO: Found 1 target...
ERROR: /path_to_edc/edc/main/BUILD:22:10: Compiling main/edc_cut.cpp failed: (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -fcolor-diagnostics -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG ... (remaining 35 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox
main/edc_cut.cpp:80:23: error: no matching constructor for initialization of 'CutMatching::Solver'
  CutMatching::Solver solver(graph.get(), subdivGraph.get(), randomGen.get(),
                      ^      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./lib/cut_matching.hpp:165:3: note: candidate constructor not viable: requires 6 arguments, but 7 were provided
  Solver(UnitFlow::Graph *g, UnitFlow::Graph *subdivGraph,
  ^
./lib/cut_matching.hpp:101:7: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 7 were provided
class Solver {
      ^
./lib/cut_matching.hpp:101:7: note: candidate constructor (the implicit move constructor) not viable: requires 1 argument, but 7 were provided
1 error generated.
Target //main:edc-cut failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 1.051s, Critical Path: 0.79s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully

I quickly go through the code and feel that the Solver initializer defined in cut_matching.cpp doesn't match the usage in
edc_cut.cpp. Is there anything that I miss?