wn-upf / Komondor

Komondor Wireless Networks Simulator
GNU General Public License v3.0
65 stars 19 forks source link

Core dumped when trying to run examples from the documentation #148

Closed ghost closed 4 years ago

ghost commented 4 years ago

First, I gave permissions to Komondor: sudo chmod -R 777 Komondor/ Next, in Komondor/Code/main, ./build_local ran succesfully Now, when I tried to do: ./komondor_main ../input/input_example/input_system_conf.csv ../input/input_example/input_nodes.csv 0 0 1 1 60 789

The console output is:


KOMONDOR Wireless Network Simulator Copyright (C) 2017-2022, and GNU GPL'd, by Sergio Barrachina & Francesc Wilhelmi

Now, assuming this was a semicolon issue in the CSV files provided already, I explicitly put semicolons after each parameter in the input_system_conf_copy.csv file (a new file). Tried to run the same command again which now results in this:


KOMONDOR Wireless Network Simulator Copyright (C) 2017-2022, and GNU GPL'd, by Sergio Barrachina & Francesc Wilhelmi

ERROR: tx_power values are not properly configured at node in line 2 MIN_TX_POWER_DBM = 1 node_container[i].tx_power_default = 0.000000 MAX_TX_POWER_DBM = 20

I have attached these csv files segmentation_fault.zip

Also note that, the files mentioned in the documentation need to be urgently updated. system_input.csv does not exist and the replacement input_system_conf.csv is located in another folder "high_density_scenarios".

fwilhelmi commented 4 years ago

Dear @oscarthelab ,

thank you for such a detailed report, this is really helpful to us.

The first issue really seems to be related to "semicolons" in the input nodes file.

Regarding the second issue, we have an input checker that tries to avoid unexpected situations from wrong input configurations. In this case, it seems that the transmission power was not properly defined since it is below the minimum value (1 dBm).

I strongly recommend using the last version of the simulator because we changed the way inputs are provided and the system configuration file is no longer needed.

I hope that helps, but let us know if you experience any other trouble.

Best regards, Francesc.

ghost commented 4 years ago

Even with the latest version, none of the execution commands as mentioned in the documentation seem to be executing properly. It would be of great help if you could just provide the direct commands to execute the examples in the docs with the current status of the csv files present in the repo. I'm running on Ubuntu 18.04 64-bit and have installed the necessary packages (i386 and others.) required to execute build_local.

fwilhelmi commented 4 years ago

Sure, given the last logs, I see that you tried to use the old "system_conf" file as nodes input. The command you need to use is as follows:

$ ./komondor_main ../input/input_example/input_nodes.csv ../output/test_output.csv 0 1 1 60 789

I hope that helps! Best, Francesc

ghost commented 4 years ago

I managed to get the simulation working only in version 2.0. Here's the command that did work : ./komondor_main ../input/input_example/input_system_conf.csv ../input/input_example/input_nodes.csv ../output/text_output.csv 1 1 1 1 0 60 789

The command mentioned by you above does not work in version 2.0 or 3.0 regardless of my inclusion of the system_conf file. Also, partial configurations don't work in either of the two versions. I've attached the error message screenshots for your reference.

kom3 0 kom2 0

It'd be great if you could look into this and also possibly update the documentation and user guide for others. Thanks for quick responses though, Shrenik

fwilhelmi commented 4 years ago

Dear @oscarthelab ,

thank you for the update. I've just tried to run the example that we have in "/input/input_example/" and worked fine. The command I used is the following:

$ ./komondor_main ../input/input_example/input_nodes.csv ../output/output.txt SIM_TEST 0 0 1 100 1234

I think last time I forgot including the "simulation_name", which now is "SIM_TEST".

I used the latest commit, where we are including many changes. Notice that we are working on a new release, but it is not yet ready for publication. In this regard, we still need to work on the documentation. Basically, we've been improving the way we introduce inputs to the simulator, so the partial input modes became deprecated. I apologize for that.

Best regards, Francesc.

ghost commented 4 years ago

Thanks, this resolved the issue.