search

psycharo-zz / factor-graph

matlab/c++ factor graph framework
32 stars 8 forks source link

warning on installation of graphlayout for duplicate assert.m #39

Closed ghost closed 11 years ago

ghost commented 11 years ago

Installation of graphlayout leads to a warning, see below. (Note: this issue may --or may not-- disappear if graphlayout were to be replaced by the newer graphviz4matlab, see #38

See the following warning after installation: 

>> installFactorGraph
Building factor graph framework for your platform...
Warning: Function
D:\github\public\factor-graph\cpp-factor-graph\matlab\ffg\graphlayout\private\assert.m
has the same name as a MATLAB builtin. We suggest you rename the function to
avoid a potential name conflict. 
> In path at 110
  In addpath at 87
  In installFactorGraph at 11 
Done!
Test suite: ffg.tests
Test suite location: Package
10-Dec-2012 16:41:00

Starting test run with 18 test cases.
..................
PASSED in 1.118 seconds.
psycharo-zz commented 11 years ago

yes it will be fixed for sure. we can simply rename the file since we have our own fork of graphlayout in the repo.