MAPIRlab / gaden

GADEN: A 3D Gas Dispersion Simulator for Mobile Robot Olfaction in Realistic Environments
GNU Lesser General Public License v3.0
60 stars 34 forks source link

gaden_demo error #8

Closed r-miko closed 6 years ago

r-miko commented 6 years ago

Hello,

When I run the launch file gaden_simulator_demo.launch I get the error: Error in opening the final output files. When I exit, it tries to write to the log file "filament_simulator1-3*.log", but I this file is not in the log directory. After running the debugger gdb on the filament simulator, I get:

screenshot from 2018-01-17 11-07-00

Also, this error may or may not be related to https://github.com/MAPIRlab/gaden/issues/7

Thank you in advance for your help, Rebecca

Huitzilo commented 6 years ago

Hi, I get this error, too... That log file doesn't exist, and the other log files in the log directory don't give away any clue about what's going wrong (except that warning about unnormalised quaternions from #7 , which doesn't really help).

We're really super-keen to replicate the results from the paper. Any ideas how to start debug this?Thanks in advance!

Huitzilo commented 6 years ago

Solved it: mkdir /<path_to_catkin_workspace>/src/gaden/gaden_demo/demo/logs_gas_dipersion_simulator/

@r-miko

See pull request #9 .

JGMonroy commented 6 years ago

Hi, This problem seems to be related to being unable to save the output log files with the simulation. I'll take a look at pull request #9 as soon as possible to create the folder if not found, but also take into account that you can change the path where to save the logs with the "/results_location" param set in the launch file: <param name="/results_location" value="$(find gaden_demo)/demo/logs_gas_dipersion_simulator" />

Huitzilo commented 6 years ago

Hi Javier,

the pull request simply creates the results directory in the github tree, so a pristine checkout should work just fine since the results directory is already in place. Thanks for the hint towards the /results_location parameter, might come in handy, too!

JGMonroy commented 6 years ago

Hi Huitzilo, Your pull request has been merged (thanks). I did not create the folder at first hand because I thought it would be better to separate the code from the results, but maybe for a first contact its better not to have errors with the demo XD. The correct way should be to set the /results_location parameter as described in the launch file.

r-miko commented 6 years ago

Hi, This works for me too. Thank you!

JGMonroy commented 6 years ago

I'm closing the issue as the problem seems solved.