JuPedSim / jpsreport

Analysis tool
https://www.jupedsim.org/jpsreport_introduction.html
Other
3 stars 9 forks source link

Utest | Methods | jpsreport failed #164

Closed gjaeger closed 4 years ago

gjaeger commented 4 years ago

JuPedSim - JPSreport

Current date : Tue Oct 15 21:29:40 2019 Version : 0.8.4 Compiler : clang++ (11.0.0) Commit hash : v0.8.4-9-ge8b96ec-dirty Commit date : Mon Oct 14 17:45:06 2019 Branch : develop Python : /usr/local/bin/python3.7 (3.7.4)

Describe the bug The example iniMethod.xml (/Utest/Methods_) crashes.

INFO:   Parsing the ini file <ini_Methods.xml>
INFO:   logfile </Users/gregorjaeger/Documents/hubs/JuPedSim_github_fork/jpsreport/Utest/Methods/logfile_Methods.txt>
INFO:   Success with Method A using measurement area id 2
INFO:   Success with Method A using measurement area id 6
Check failed in file /usr/local/include/boost/numeric/ublas/functional.hpp at line 1418:
i < size_i
libc++abi.dylib: terminating with uncaught exception of type boost::numeric::ublas::bad_index: bad index
zsh: abort       ini_Methods.xml
chraibi commented 4 years ago

Could you pinpoint the line where this crash happens?

Maybe some id in the inifile is wrong?

gjaeger commented 4 years ago

The problem lies in method B. If method B is not called, jpsreport runs.

<measurement_areas unit="m">

    <area_B id="4" type="BoundingBox" zPos="None">
      <vertex x="20" y="12" />
      <vertex x="24" y="12" />
      <vertex x="24" y="8" />
      <vertex x="20" y="8" />
      <length_in_movement_direction distance="4" />
    </area_B>

    <area_B id="7" type="BoundingBox" zPos="None">
      <vertex x="24" y="12" />
      <vertex x="28" y="12" />
      <vertex x="28" y="8" />
      <vertex x="24" y="8" />
      <length_in_movement_direction distance="4" />
    </area_B>

  </measurement_areas>
  <method_B enabled="true">
    <measurement_area id="4" />
    <measurement_area id="7" />
  </method_B>

I don't find a bug in the ini-file. I'm reviewing my changes to pull request #161 again.

gjaeger commented 4 years ago

@chraibi Can it be related to the change in commit fc944d5c6? It's the last major change in Method B.