search

iut-ibk / DynaMind-ToolBox

DynaMind-ToolBox
www.dance4water.org
GNU General Public License v2.0
9 stars 6 forks source link

Python Modules not loaded #179

Closed christianurich closed 11 years ago

christianurich commented 11 years ago

Traceback (most recent call last): File "/Users/christianurich/Documents/DynaMind-ToolBox/build/debug/output/pydynamind.py", line 5163, in createNode return self.klass().disown() File "/Users/christianurich/Documents/DynaMind-ToolBox/build/debug/output/PythonModules/scripts/Modules/ExportToShapeFile.py", line 78, in init self.createParameter("FileName", STRING, "test") File "/Users/christianurich/Documents/DynaMind-ToolBox/build/debug/output/pydynamind.py", line 5055, in createParameter self.addParameter(name,DN_type,self._data[name],description) File "/Users/christianurich/Documents/DynaMind-ToolBox/build/debug/output/pydynamind.py", line 5006, in addParameter return _pydynamind.Module_addParameter(self, args) NotImplementedError: Wrong number or type of arguments for overloaded function 'Module_addParameter'. Possible C/C++ prototypes are: Module::addParameter(std::string const &,DataTypes const,void ,std::string const) Module::addParameter(std::string const &,DataTypes const,void *)

zacharias2k commented 11 years ago

self.createParameter("FileName", STRING, "test") "test" is not a pointer? - can you convert it to one?

christianurich commented 11 years ago

That is not the problem. parameter is a python function in pydynamind.i that calls addParamter

self.addParameter(name,DN_type,self._data[name],description)

A bug fix was that worked for me was to change the data type into an integer again.

zacharias2k commented 11 years ago

I will redo the parameter stuff in near future anyway. It's still tricky and not typesafe.

christianurich commented 11 years ago

Should I commit my changes in the meanwhile. I would really like to use python modules as well On 6 Aug 2013 18:08, "zacharias2k" notifications@github.com wrote:

I will redo the parameter stuff in near future anyway. It's still tricky and not typesafe.

— Reply to this email directly or view it on GitHubhttps://github.com/iut-ibk/DynaMind-ToolBox/issues/179#issuecomment-22163654 .

zacharias2k commented 11 years ago

yes, but add a // FIX comment in the source, just to get sure it is documented.

abroxos commented 11 years ago

I'll check this

christianurich commented 11 years ago

potential bug fix see https://github.com/iut-ibk/DynaMind/pull/33

abroxos commented 11 years ago

@christianurich Tested with ExportToShape -> wrong nameing of enum :-) Please pull, check and close.

christianurich commented 11 years ago

awesome :-)