Karim Sabri / ctools

# Conflicts:
#	ChangeLog
#	NEWS

# Conflicts:
#	ChangeLog
#	NEWS
#	README.md
#	configure.ac
#	ctools.pc.in
#	doc/Doxyfile
#	doc/source/admin/release_history/1.3.1.rst
#	doc/source/conf.py
#	sonar-project.properties
#	src/ctmodel/ctmodel.cpp

Changed the on time for a given bin to be set from the observation being filled into the cube rather than the cube itself.
Reformat code (#2147)

Changed the on time for a given bin to be set from the observation being filled into the cube rather than the cube itself.
Reformat code (#2147)

Optimized computation intensive parts of ctmodel to reduce computation time.
Implemented openmp parallelization of ctmodel to reduce computation time.
Updated information logging (#2147)

Updated writing information to log file in order to prevent messy log file messages. This doesnt appear to impact the computation time.
Integrate code without modification of GammaLib (#2147)
Implement efficient model computation (#2147)
Update change log (#2147)

Optimized computation intensive parts of ctmodel to reduce computation time.
Implemented openmp parallelization of ctmodel to reduce computation time.
Updated information logging (#2147)

Updated writing information to log file in order to prevent messy log file messages. This doesnt appear to impact the computation time.
Integrate code without modification of GammaLib (#2147)
Implement efficient model computation (#2147)

Removed dynamic scheduling as it introduces a small amount of overhead (it was only done so that the observations would be analyzed in numerical order)
Added caching of sky directions for each pixel to prevent the need to re-convert the positions from index for each observation.
Fill cached sky positions with RA,Dec values to prevent need to convert between galactic->celestial every time the distance is compute a distance from an observation ROI.

Updated headers in log file to reflect what is actually going on.
Changed omp scheduling to 'dynamic' so each observation is filled sequentially.

Finalized a workaround to the python wrapper issue that caused the 'ctbin::run()' method to crash.

Added check on whether observation is null or a CountsCube, which was done automatically before when calling 'next_unbinned_observation', but needs to be done explicitly now.

Added critical region when incrementing the bins of the counts cube. This didn't impact the computation time, but may improve the reliability of filling the cube if the same bin is being filled by two different observations.

Parallelized computation over different observations.

The working log file pointer was not forwarded to ctobssim::set_mc_id_names() and consequently the standard log file pointer was used, which occasionally lead to the exception during the Python unit test. Now the log file pointer is forwarded properly.

ctexpcube was in fact executed twice due to a bug in the main.cpp file.

# Conflicts:
#	src/ctselect/ctselect.par

# Conflicts:
#	src/Makefile.am