JPScore issueshttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues2018-06-13T16:29:19+02:00https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/64Documentation issues2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.deDocumentation issues1. page 22: geometry-file: missing "unit" entry
2. page 23: it is not obvious that subrooms are mandatory
3. routing-file: hint that either triangulation or hlines can be used in a project.
4. page 7: provide the location of the definiti...1. page 22: geometry-file: missing "unit" entry
2. page 23: it is not obvious that subrooms are mandatory
3. routing-file: hint that either triangulation or hlines can be used in a project.
4. page 7: provide the location of the definition files
5. page 25: inconsistencies between crossings and transitionshttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/63Agents parameters (velocities) on stairs2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.deAgents parameters (velocities) on stairsDifferent stairs types (staircase, escalator) ?Different stairs types (staircase, escalator) ?https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/62option for showing the statistics in the xml input file2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.deoption for showing the statistics in the xml input file<show_statistics> </show_statistics><show_statistics> </show_statistics>https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/61VS2013 fails to compile after push2018-03-16T15:01:07+01:00Mohcine Chraibim.chraibi@fz-juelich.deVS2013 fails to compile after pushFehler 97 error C4716: 'Room::SetEgressTime': Muss einen Wert zurückgeben c:\users\oliver\documents\jpscore\geometry\room.cpp 133 1 core
Fehler 136 error C4716: 'QuickestPathRouter::GetTravelTime': Muss einen Wert zurückgeben c:\users\ol...Fehler 97 error C4716: 'Room::SetEgressTime': Muss einen Wert zurückgeben c:\users\oliver\documents\jpscore\geometry\room.cpp 133 1 core
Fehler 136 error C4716: 'QuickestPathRouter::GetTravelTime': Muss einen Wert zurückgeben c:\users\oliver\documents\jpscore\routing\quickestpathrouter.cpp 635 1 core
methods should be removed if not used, or return a value, or set to void.
It is 'only' a warning in g++ but an error in vs2013https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/60runtest_1 to x fail under VS (except of 11, there must be another bug too)2018-06-13T16:29:01+02:00Mohcine Chraibim.chraibi@fz-juelich.deruntest_1 to x fail under VS (except of 11, there must be another bug too)Starting a regression test with visual studio showed that every single runtest fail.
The reason is path incompability between windows and unix.
e.g.
C:\Users\Oliver\Documents\jpscore\Utest\test_10/master_ini.xml
should be
C:\Users\Olive...Starting a regression test with visual studio showed that every single runtest fail.
The reason is path incompability between windows and unix.
e.g.
C:\Users\Oliver\Documents\jpscore\Utest\test_10/master_ini.xml
should be
C:\Users\Oliver\Documents\jpscore\Utest\test_10\master_ini.xml
The bug is in the python testdriver.
For instance there is this code on line 54
geofile = "%s/geometry.xml"%DIR
which should be
geofile = os.path.join(DIR, "geometry.xml")
to be platform independent.
ALL python testdrivers have to be checked and fixed, and after that testet on Windows, Linux and OSX again.
However runtest_11 says it passed although it never run with the correct inputfiles, so there has to be another different bug in it. Please check.
@chraibi this is also important in other projects like JPSreporthttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/59Unittests fail to compile with VS 20132018-03-16T15:01:07+01:00Mohcine Chraibim.chraibi@fz-juelich.deUnittests fail to compile with VS 2013Fehler 145 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testClassPoint.cpp 39 1 testClassPoint
Fehler 146 error C1083: Datei (Include) kann nicht g...Fehler 145 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testClassPoint.cpp 39 1 testClassPoint
Fehler 146 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testClassLine.cpp 38 1 testClassLine
Fehler 147 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testEffectiveDistanceToEllipse.cpp 39 1 testEffectiveDistanceToEllipse
Fehler 148 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testEffectiveDistanceToLine.cpp 39 1 testEffectiveDistanceToLine
Fehler 149 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testGetArea.cpp 39 1 testGetArea
Fehler 150 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testIsInside.cpp 39 1 testIsInside
Fehler 151 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testIsOn.cpp 39 1 testIsOn
Fehler 152 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testIsOutside.cpp 39 1 testIsOutside
Fehler 153 error C1083: Datei (Include) kann nicht geöffnet werden: "unistd.h": No such file or directory C:\Users\Oliver\Documents\jpscore\Utest\testPointOnEllipse.cpp 39 1 testPointOnEllipse
reason is that unistd.h is only available for unix systems, maybe we can avoid this include?
Edit:
#ifdef WINDOWS
#include < direct.h >
#define GetCurrentDir _getcwd
#else
#include < unistd.h >
#define GetCurrentDir getcwd
#endif
WINDOWS seems not to be defined although i definitly use windows. maybe there is an error in cmake?Mohcine Chraibim.chraibi@fz-juelich.deMohcine Chraibim.chraibi@fz-juelich.dehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/58Results of Codeanalysis by Visual Studio, possible bugs and errors2018-03-16T15:01:07+01:00Mohcine Chraibim.chraibi@fz-juelich.deResults of Codeanalysis by Visual Studio, possible bugs and errorsthis should be fixed as soon as possible or if unused removed. sry for german error massages.
C6387 Ungültiger Parameterwert 'str' könnte '0' sein: dies entspricht nicht der Spezifikation für Funktion 'atoi'. core argumentparser.h 237...this should be fixed as soon as possible or if unused removed. sry for german error massages.
C6387 Ungültiger Parameterwert 'str' könnte '0' sein: dies entspricht nicht der Spezifikation für Funktion 'atoi'. core argumentparser.h 237
'str' kann NULL sein. (In diese Verzweigung eintreten) 237
'str' ist eine Eingabe für 'atoi' (in c:\program files (x86)\microsoft visual studio 12.0\vc\include\stdlib.h:430 deklariert) 237
'str' sollte nicht NULL sein, da dies nicht der SAL-Anmerkung zu 'atoi' entspricht. 237
C6387 Ungültiger Parameterwert 'str' könnte '0' sein: dies entspricht nicht der Spezifikation für Funktion 'atof'. core argumentparser.h 244
'str' kann NULL sein. (In diese Verzweigung eintreten) 244
'str' ist eine Eingabe für 'atof' (in c:\program files (x86)\microsoft visual studio 12.0\vc\include\math.h:527 deklariert) 244
'str' sollte nicht NULL sein, da dies nicht der SAL-Anmerkung zu 'atof' entspricht. 244
C6031 Rückgabewert wird ignoriert Rückgabewert wird ignoriert: 'getc'. core directionstrategy.cpp 108
C6031 Rückgabewert wird ignoriert Rückgabewert wird ignoriert: 'getc'. core directionstrategy.cpp 114
/*CRITICAL
C6011 Dereferenzierender NULL-Zeiger. Dereferenzierender NULL-Zeiger 'act_cell'. core meshrouter.cpp 581
'act_cell' kann NULL sein. 578
Diese Schleife überspringen (Annahme: '(bool)=i<*.size(), (), (bool)' ist false) 581
'act_cell' wird dereferenziert, kann jedoch weiterhin NULL sein. 581
/*CRITICAL
C28182 Dereferenzieren der Kopie eines NULL-Zeigers Dereferenzierender NULL-Zeiger. 'sub' enthält den gleichen NULL-Wert wie 'stair'. core navmesh.cpp 2471
'sub' kann NULL sein. (In diese Verzweigung eintreten) 2459
In diese Schleife eintreten (Annahme: '<Verzweigungsbedingung>') 2463
In diese Schleife eintreten (Annahme: 'k<r->GetNumberOfSubRooms()') 2465
In diese Verzweigung eintreten (Annahme: 'st!=0') 2469
'sub' wird dereferenziert, kann jedoch weiterhin NULL sein. 2471
C6031 Rückgabewert wird ignoriert Rückgabewert wird ignoriert: 'getc'. core pedestrian.cpp 666
/*CRITICAL
C28182 Dereferenzieren der Kopie eines NULL-Zeigers Dereferenzierender NULL-Zeiger. 'hline' enthält den gleichen NULL-Wert wie 'cros=dynamic_cast<Crossing *>hline, cros'. In Zeile 435 finden Sie einen früheren Ort, an dem dies vorkommen kann. core quickestpathrouter.cpp 443
'hline' wird dereferenziert, kann jedoch weiterhin NULL sein. 443
/*CRITICAL
C28182 Dereferenzieren der Kopie eines NULL-Zeigers Dereferenzierender NULL-Zeiger. 'hline' enthält den gleichen NULL-Wert wie 'cros=dynamic_cast<Crossing *>hline, cros'. In Zeile 524 finden Sie einen früheren Ort, an dem dies vorkommen kann. core quickestpathrouter.cpp 534
'hline' wird dereferenziert, kann jedoch weiterhin NULL sein. 534https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/57compilation with xcode generator2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.decompilation with xcode generator@kemloh: someone has to check this.@kemloh: someone has to check this.https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/56compilation with Visual Studio 12 (2013) generator2018-03-16T15:01:07+01:00Mohcine Chraibim.chraibi@fz-juelich.decompilation with Visual Studio 12 (2013) generatorI tried to compile again under windows with cmake-gui and the Visual Studio 2013 generator. CMake gives me the following output:
JPSCORE_VERSION: 0.5.0
BUILD_TESTING: OFF
PROCESSOR_COUNT: 8
CMAKE_BUILD_TYPE: Release
CMAKE_BINARY_DIR: C:...I tried to compile again under windows with cmake-gui and the Visual Studio 2013 generator. CMake gives me the following output:
JPSCORE_VERSION: 0.5.0
BUILD_TESTING: OFF
PROCESSOR_COUNT: 8
CMAKE_BUILD_TYPE: Release
CMAKE_BINARY_DIR: C:/Users/Oliver/Documents
CMAKE_SOURCE_DIR: C:/Users/Oliver/Documents/jpscore
Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
Checking for C++11 compiler - unavailable
Performing Test COMPILER_SUPPORTS_CXX11
Performing Test COMPILER_SUPPORTS_CXX11 - Failed
Performing Test COMPILER_SUPPORTS_CXX0X
Performing Test COMPILER_SUPPORTS_CXX0X - Failed
The compiler C:/Program Files (x86)/Microsoft Visual Studio 12.0/VC/bin/cl.exe has no C++11 support. Please use a different C++ compiler.
Release flags: /DWIN32 /D_WINDOWS /W3 /GR /EHsc /openmp , /MD /O2 /Ob2 /D NDEBUG
Configuring done
It says there is no c++11 support. However there is.
After generating the files i opened the solution file (*.sln) for visual studio. If i try to gernate this solution there are a total of 19 errors and 120 warnings. I post the errors. Most of them occur because of header files which are not found.
Fehler 1 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 2 error C1083: Datei (Include) kann nicht geöffnet werden: "getopt.h": No such file or directory C:\Users\Oliver\Documents\jpscore\general\ArgumentParser.cpp 28 1
Fehler 3 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 90 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 546 1
Fehler 91 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 588 1
Fehler 92 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 822 1
Fehler 93 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 837 1
Fehler 94 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 877 1
Fehler 95 error C3016: "p": Die Indexvariable in der For-Anweisung von OpenMP muss einen ganzzahligen Typ mit Vorzeichen aufweisen. C:\Users\Oliver\Documents\jpscore\math\GPU_GCFMModel.cpp 928 1
Fehler 120 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 121 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 122 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 123 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
Fehler 134 error C1083: Datei (Include) kann nicht geöffnet werden: "bits/random.h": No such file or directory c:\users\oliver\documents\jpscore\routing\GlobalRouter.h 40 1
135 IntelliSense: Die Datei "Quelle" kann nicht geöffnet werden: "getopt.h". c:\Users\Oliver\Documents\jpscore\general\ArgumentParser.cpp 28 1 core
136 IntelliSense: Die Datei "Quelle" kann nicht geöffnet werden: "unistd.h". c:\Users\Oliver\Documents\jpscore\general\ArgumentParser.cpp 29 1 core
137 IntelliSense: Die Datei "Quelle" kann nicht geöffnet werden: "bits/random.h". c:\Users\Oliver\Documents\jpscore\routing\GlobalRouter.h 40 1 corehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/55Python 3.x tests2018-06-13T16:29:01+02:00Mohcine Chraibim.chraibi@fz-juelich.dePython 3.x testsCheck and if necessary change the py-scripts (in Utest) such that they run with python 3Check and if necessary change the py-scripts (in Utest) such that they run with python 3https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/54plane equation variables are misleading.2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.deplane equation variables are misleading.Change to A_x, B_y, D
now they are A_x, B_y, C_z which might be misleading.
The expliyit equation is given by Z= Ax+ By+DChange to A_x, B_y, D
now they are A_x, B_y, C_z which might be misleading.
The expliyit equation is given by Z= Ax+ By+Dhttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/53Provide more checks for the geometry and at runtime2018-06-13T16:29:01+02:00Mohcine Chraibim.chraibi@fz-juelich.deProvide more checks for the geometry and at runtimeSample rules could be:
* no overlapping between doors and walls
* closed rooms (walls only)
* overlapping rooms/areas
* agents caught in obstacles
* agent goes through wall
* ...
to be implemented in the SanityCheck() functionSample rules could be:
* no overlapping between doors and walls
* closed rooms (walls only)
* overlapping rooms/areas
* agents caught in obstacles
* agent goes through wall
* ...
to be implemented in the SanityCheck() functionhttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/52Update files version to v0.62018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.deUpdate files version to v0.6Update all files version to v0.6 in preperation of the next releaseUpdate all files version to v0.6 in preperation of the next releasehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/51jpscore_compile failes when in source compiled2018-06-13T16:29:19+02:00Mohcine Chraibim.chraibi@fz-juelich.dejpscore_compile failes when in source compiledSome directories in CMakeLists.txt are not correctly set
```
--build-exe-dir ${EXECUTABLE_OUTPUT_PATH}
```Some directories in CMakeLists.txt are not correctly set
```
--build-exe-dir ${EXECUTABLE_OUTPUT_PATH}
```Mohcine Chraibim.chraibi@fz-juelich.deMohcine Chraibim.chraibi@fz-juelich.dehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/49more statistics: evacuation time for rooms2018-06-13T16:29:18+02:00Mohcine Chraibim.chraibi@fz-juelich.demore statistics: evacuation time for roomsgenerate more statistics at the end of the simulation.
Option to display them can be specified from the inputfilegenerate more statistics at the end of the simulation.
Option to display them can be specified from the inputfilehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/48moscawa ini-files2018-06-13T16:29:20+02:00Mohcine Chraibim.chraibi@fz-juelich.demoscawa ini-filesThe following ini-files in inputfiles/moscawa:
1. ini-mall.xml
2. and ini.xml
have some router-issues. (duplicate router found)
The file ini-T.xml however is working fine.
Maybe this directory needs some cleaning?The following ini-files in inputfiles/moscawa:
1. ini-mall.xml
2. and ini.xml
have some router-issues. (duplicate router found)
The file ini-T.xml however is working fine.
Maybe this directory needs some cleaning?https://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/47Default files2018-06-13T16:29:18+02:00Mohcine Chraibim.chraibi@fz-juelich.deDefault filesrun `jpscore` without parameters should consider using the default files in `inputfiles/Default`run `jpscore` without parameters should consider using the default files in `inputfiles/Default`Mohcine Chraibim.chraibi@fz-juelich.deMohcine Chraibim.chraibi@fz-juelich.dehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/46Uniform distribution of pedestrians2018-03-16T15:01:07+01:00Mohcine Chraibim.chraibi@fz-juelich.deUniform distribution of pedestriansMohcine Chraibim.chraibi@fz-juelich.deMohcine Chraibim.chraibi@fz-juelich.dehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/44Turning parameter2018-06-13T16:29:18+02:00Mohcine Chraibim.chraibi@fz-juelich.deTurning parameterMohcine Chraibim.chraibi@fz-juelich.deMohcine Chraibim.chraibi@fz-juelich.dehttps://gitlab.jsc.fz-juelich.de/jupedsim/jpscore/-/issues/43Events management2018-06-13T16:29:18+02:00Mohcine Chraibim.chraibi@fz-juelich.deEvents managementAnkeAnke