Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make SWIG-MATLAB work for Octave #67

Closed
6 tasks done
jaeandersson opened this issue Jun 13, 2016 · 4 comments
Closed
6 tasks done

Make SWIG-MATLAB work for Octave #67

jaeandersson opened this issue Jun 13, 2016 · 4 comments

Comments

@jaeandersson
Copy link
Owner

jaeandersson commented Jun 13, 2016 •
edited
Loading

Octave has support for most of the features used by the MATLAB SWIG module: mex, classdef etc. It should (in principle at least) be possible to make the generated SWIG-MATLAB modules compilable for both Octave and MATLAB. This would be an attractive alternative to using SWIG's Octave module in many cases.

Issues encountered:

  • .cxx ending not recognised by mkoctfile
  • stdio.h missing (needed by vsnprintf)
  • CallMATLABWithTrap missing
  • mxSetProperty, mxGetProperty missing
  • ne not defined for handle objects
  • import module.* is not supported by Octave
jaeandersson added a commit that referenced this issue Jun 13, 2016
@jaeandersson
[MATLAB] Issue #67 Added stdio.h include
9f00110
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Eliminated use of mxGetProperty, mxSetProperty
11d8b29
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Fixed class example
c0d3df4
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Workaround for missing 'ne' for Octave handle
8deedaf
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Conditional compilation to avoid mexCallMATLABWith…
fc3c282 
…Trap in Octave
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Made swigPtr public (to avoid test failure)
0763f39
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 Avoiding mexCallMATLABWithTrap in director
4b62f60
jaeandersson added a commit that referenced this issue Jun 14, 2016
@jaeandersson
[MATLAB] Issue #67 SWIG_Matlab_CallInterp(Ex) helper functions
5d3770e
@jaeandersson jaeandersson mentioned this issue Jun 14, 2016
Closed
@jaeandersson
Copy link
Owner Author

Pushed to the "matlab" branch. Note that this adds one MATLAB file: SwigGet.m. You'd need to distribute this file for the updated module to work. But then it should work for both MATLAB and Octave.

jaeandersson added a commit that referenced this issue Jun 21, 2016
@jaeandersson
[MATLAB] Issue #67 Using Octave to test MATLAB module
700c68a
jaeandersson added a commit that referenced this issue Jun 21, 2016
@jaeandersson
[MATLAB] Issue #67 Fixed unit tests
ae34f62
traversaro added a commit to robotology/idyntree that referenced this issue Jun 24, 2016
@traversaro
[matlab] Install new file SwigGet.m introduced in jaeandersson/swig#6…
c8fb9d7 
…7 (comment)
jaeandersson added a commit that referenced this issue Jun 28, 2016
@jaeandersson
Issue #6 #67 Fixed unit tests
d21cdd1
jaeandersson added a commit that referenced this issue Jun 29, 2016
@jaeandersson
[MATLAB] Issue #67 Fixed unit tests
6875667
jaeandersson added a commit that referenced this issue Jun 29, 2016
@jaeandersson
Issue #6 #67 Fixed unit tests
a8b005e
@jaeandersson
Copy link
Owner Author

Alerted octave devs about the fact that mkoctfile doesn't recognize ".cxx" suffix: https://savannah.gnu.org/bugs/index.php?48381. Can't do much more than that.

jaeandersson added a commit that referenced this issue Jul 3, 2016
@jaeandersson
Issue #67 Avoiding import in test suite
b5e75d1 
Not supported in Octave and bad style in Matlab
jaeandersson added a commit that referenced this issue Jul 3, 2016
@jaeandersson
Issue #67 Tried to resolve Octave mex.h failure
61b7c6f
@jaeandersson
Copy link
Owner Author

Octave issue fixed: ".cxx" will be recognized by mkoctfile in Octave 4.2.

jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 MATLAB/Octave mex compilation
54733ce
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Replaced bool with int
95fe8da
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Fixed case insensitive conflict in test
17c2548
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Fixed rename_scope_runme.m
204403d
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Fixed enums.i
dd3b60c
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Fixed preproc.i
8f0eafc
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 nested_structs.i
8faf06a
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Simplified MATLAB/Octave build system
dea4ffb
jaeandersson added a commit that referenced this issue Jul 5, 2016
@jaeandersson
Issue #67 Added missing dependency on limits.h
f7b2dc6
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Tried to resolve Octave mex.h failure
2ae4e76
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Replaced bool with int
7f71bff
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Fixed case insensitive conflict in test
45c4886
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Fixed rename_scope_runme.m
1ce973e
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Fixed enums.i
288a323
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Fixed preproc.i
ff091e9
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 nested_structs.i
abe6b99
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Added missing dependency on limits.h
e9a5a52
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Added list of MATLAB keywords
38af59f
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Buildsystem cleanup
c1c36cc
jaeandersson added a commit that referenced this issue Jul 7, 2016
@jaeandersson
Issue #67 Made build system MATLAB/Octave bilingual
7e057d2
@jaeandersson
Copy link
Owner Author

Build system now bilingual: Set the "--enable-octave-for-matab" configure option to use Octave instead of MATLAB for the testing (the generated code code is always bilingual). One unit test fails for Octave: #69.

jaeandersson added a commit that referenced this issue Jul 8, 2016
@jaeandersson
Issue #67 'end' must sometimes be overloaded
090946a
kjyv pushed a commit to robotology/idyntree that referenced this issue Jul 14, 2016
Merge branch 'master' into jointTorqueRegressor
232bec9 
* master: (40 commits)
  Update README.md
  [bindings] update matlab bindings
  Updataed SixAxisFTOffsetEstimatiom matlab examples
  [estimation] Add addNewUnknownFullWrenchInFrameOrigin to LinkUnknownWrenchContacts
  [tools] Added --print (-p) option to idyntree-model-info
  Added option in urdf parser to threat sensor frames as additional frames
  [estimation] Exposed momentum derivative information in estimator and added test
  Expose estiamtion of net wrench acting on a link in ExtWrenchesAndJointTorquesEstimator class
  Update model_loading.md
  Create model_loading.md
  [cmake] Set default value of IDYNTREE_DOXYGEN_VERBOSE to OFF
  Update faqs.md
  Update GetJointAxesInWorldFrame.m
  [matlab] Regenerate bindings
  [matlab] Install new file SwigGet.m introduced in jaeandersson/swig#67 (comment)
  Update README.md
  [matlab] Updated wrapping of joints in Bindings In particular, add helper methods in IJoint interface to overcome the lack of dynamic_cast-like operator in scripting languages.
  Working toward exposing revolute joints method in bindigs
  Fixed angular velocity projection to sensor frame in predictMeasurements
  Fixed angularAccelerationTest in PredictSensorsMeasurementsUnitTest
  ...
@traversaro traversaro mentioned this issue May 29, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jaeandersson