Commit Graph

10357 Commits (b8de033bd0510367a044d0fddfa32127d7d61819)

Author SHA1 Message Date
Frank Dellaert 61241ee9ff fixed expected wrapper file 2017-12-02 22:14:56 -08:00
Frank Dellaert e8e2e52c94 Fixed test 2017-12-02 18:55:44 -08:00
Frank Dellaert 4188a739ec Fixed overloaded methods/constructors 2017-12-02 18:43:18 -08:00
Frank Dellaert 0dd7dcdc9f Merge branch 'develop' into feature/cython_wrapper 2017-12-02 18:33:02 -08:00
Frank Dellaert 661fedbfb0 commented, verified fix with Duy 2017-12-02 17:20:27 -08:00
Frank Dellaert 843682cd7e Merge branch 'develop' into feature/fixExpressionFactorKeys 2017-12-02 15:55:25 -08:00
Ignacio Vizzo 9f09069b65 Merged in IgnacioVizzo/fix-win32_build_error_ccolambd (pull request #302)
Fix win32 build error for 3rdparty/ccolamd module

Approved-by: Ignacio Vizzo <ivizzo@irobot.com>
2017-10-27 17:31:11 +00:00
Ignacio Vizzo 4ebe9ec2dc Fix win32 build error for 3rdparty/ccolamd module
Supposing there is a typo in this line:
\# make sure that ccolamd compiles even in face of warnings
if(WIN32)
  set_source_files_properties(${3rdparty_srcs} PROPERTIES COMPILE_FLAGS "/w")

This will add "/w" to the compiler line, and on windows(gcc 6.3.0)
this will produce a build error:
  "gcc.exe: error: W:/: No such file or directory"

This change was introduced in gtsam with this commit:

  commit 7e2e0aaa6d
  Author: Frank <frank@skyd.io>
  Date:   Tue Mar 1 15:41:28 2016 -0800

   Appended flags instead of overwriting
2017-10-25 19:15:35 -03:00
Chris Beall ac1e516037 Merged in feature/CCOLAMD-BSD (pull request #301)
CCOLAMD now BSD-3
2017-10-17 18:40:22 +00:00
chrisbeall 18fb907a85 As of SuiteSparse 4.5.6 CCOLAMD is now licensed under BSD-3! 2017-10-17 08:27:44 -07:00
Duy-Nguyen Ta ee176b7669 fix a typo in requirements.txt 2017-10-13 21:59:33 -04:00
Duy-Nguyen Ta 6d48214a9f wrap RangeFactorWithTransform 2017-10-08 22:12:29 -04:00
Duy-Nguyen Ta 0d47d273c6 Fix the dependency problem where cython didn't get called when the header file is updated. Also change target gtsam_cython to cythonize_gtsam. 2017-10-08 22:11:54 -04:00
Duy-Nguyen Ta 3e8807b575 fix constness 2017-10-08 22:06:44 -04:00
Duy-Nguyen Ta 7ecdbd5908 update expected result for cython wrap test 2017-08-28 14:12:15 -04:00
Duy-Nguyen Ta d23b5e4cfd only catch AssertionError exceptions when handling overloads, so that other C++ exceptions can be raised correctly 2017-08-25 11:18:16 -07:00
Duy-Nguyen Ta 9325fda568 Merged in feature/cython_wrapper-build_improvements (pull request #299)
Feature/cython wrapper build improvements

Approved-by: Chris Beall <chrisbeall@gmail.com>
2017-08-18 01:06:21 +00:00
Duy-Nguyen Ta b74875f10a Similar to Matlab wrapper, CYTHON_TOOLBOX and POINTS_TO_VECTOR can't co-exist for now 2017-08-17 07:03:35 -04:00
Duy-Nguyen Ta 005796f209 remove bad committed cmake files 2017-08-16 22:21:41 -04:00
Duy-Nguyen Ta 6b208c0340 fix eigency build for Linux. Only Mac OS's linker needs the dynamic_lookup option for undefined symbols 2017-08-16 22:13:38 -04:00
Frank Dellaert 03c939fafe Merged in feature/faster_cython (pull request #300)
Faster and more concise cython

Approved-by: Duy-Nguyen Ta <thduynguyen@gmail.com>
2017-08-15 22:15:41 +00:00
Duy-Nguyen Ta b8cd7da08b fix gtsam.h to compile with matlab 2017-08-15 16:39:25 -04:00
Duy-Nguyen Ta da5d3e303c update expected outputs for cython wrapper unittests 2017-08-15 13:46:32 -04:00
Duy-Nguyen Ta 356f47a123 proper installation of gtsam_eigency's __init__.py 2017-08-15 12:33:42 -04:00
Duy-Nguyen Ta 973b2337a2 make unstable build again 2017-08-15 11:16:13 -04:00
Duy-Nguyen Ta acad9b0339 disable all warnings while compiling cython generated cpp 2017-08-14 17:23:24 -04:00
Duy-Nguyen Ta 6fb520b89d make cython fail at the first error 2017-08-14 17:22:04 -04:00
dellaert 16057a2c8e Unit test, succeeds... 2017-08-08 00:59:33 -07:00
dellaert 45a7b5ba68 Add edge/vertex parsing from any input stream 2017-08-08 00:39:22 -07:00
dellaert 3635380be4 Re-factored and re-formatted, incorporated Duy's fix (don't de-reference bayesTree) 2017-08-07 21:35:24 -07:00
dellaert d712498c08 Fixed this test with V4 compatibility (where Points are not initialized) 2017-08-07 08:37:37 -07:00
dellaert c5a0f1a839 Undo unrelated changes 2017-08-06 17:42:20 -07:00
dellaert 3bbea0f301 Make pxd a bit nicer to read 2017-08-06 17:26:12 -07:00
dellaert 905aac29f8 Only define as many return values as needed 2017-08-06 17:15:47 -07:00
dellaert 81c15d950a Removed kwargs overhead for overloaded methods 2017-08-06 16:53:04 -07:00
dellaert 8ae9d7c577 Added more tests 2017-08-06 15:59:24 -07:00
dellaert d752c9e249 Re-wrote constructor overloading logic which saves a lot of overhead 2017-08-06 14:58:23 -07:00
dellaert e02b504575 Constructor overloads are supported, but arguments *have* to have different names 2017-08-06 14:57:40 -07:00
dellaert 2374347e69 Make internal, overloaded static methods cdefs to avoid call code/overhead 2017-08-06 13:25:54 -07:00
dellaert 9505144f44 More unit tests, specifically one that exercises overloaded static method 2017-08-06 13:25:00 -07:00
dellaert 3dfb13d3d6 Added missing Rot3 constructors 2017-08-06 13:23:29 -07:00
dellaert 6068166d2b Moved wrapper class signature to Class.cpp 2017-08-06 12:21:26 -07:00
dellaert 74a33ff222 Re-structured argument overloading to call a common function 2017-08-06 11:07:13 -07:00
dellaert 81bb1d445a Fixed cmake so that it really defines CYTHON_EXECUTABLE (instead of just threatening to do so). This has the side-benefit that compilation now works from within eclipse. 2017-08-05 16:30:20 -07:00
Frank Dellaert c82fe1fde2 Initial Bitbucket Pipelines configuration 2017-08-05 20:20:16 +00:00
Duy-Nguyen Ta 17c4c2ff19 don't install cmake and config files 2017-07-28 15:48:30 -04:00
Duy-Nguyen Ta 82531c561f clonedEigency --> gtsam_eigency. Update readme. 2017-07-28 15:26:19 -04:00
Duy-Nguyen Ta 931405155e move GTSAM_CYTHON_INSTALL_PATH to main CMakeLists. Set eigency install path in gtsam_extra.cmake so other non-gtsam projects can find. 2017-07-28 15:24:30 -04:00
Duy-Nguyen Ta b73e22220d remove FindClonedEigency.cmake 2017-07-28 15:22:40 -04:00
Duy-Nguyen Ta e7e75c3ac2 update readme 2017-07-27 22:44:16 -04:00