Error building MATLAB Bindings

Open
#19 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
3/5
Estimated time
1-2 days
Newbie friendliness
25/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
cpp, matlab

Research direction

Start with clipper/bindings/matlab/finddensecluster_mex.cpp and trace the declarations for clipper::utils::randvec and clipper::findDenseCluster in the C++ sources. Reproduce the MATLAB binding build on Ubuntu 22.04; done means the binaries compile without these missing-declaration errors.

Written by the indexing model from the issue text.

Description

Hello,

I'm trying to compile the MATLAB binaries on Ubuntu 22.04 and I get the following errors:

clipper/bindings/matlab/finddensecluster_mex.cpp:65:37: error: ‘clipper::utils’ has not been declared
   65 |   if (u0.size() == 0) u0 = clipper::utils::randvec(m);
      |                                     ^~~~~
clipper/bindings/matlab/finddensecluster_mex.cpp:74:21: error: ‘findDenseCluster’ is not a member of ‘clipper’
   74 |     soln = clipper::findDenseCluster(Ms, Cs, u0, params);
      |                     ^~~~~~~~~~~~~~~~
clipper/bindings/matlab/finddensecluster_mex.cpp:76:21: error: ‘findDenseCluster’ is not a member of ‘clipper’
   76 |     soln = clipper::findDenseCluster(M, C, u0, params);
      |                     ^~~~~~~~~~~~~~~~

On inspection, it seems findDenseCluster is not a declared function in the C++ code. Am I missing something? How did you compile the MATLAB binaries?

Dominant language
C++
Stars
289
Forks
45
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from mit-acl/clipper

All issues in mit-acl/clipper

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.