Justine Devin / gammalib

set_mc_cone() called fetch_cube() which itself called load() which itself called set_mc_cone() with a larger cone again. This led to a neverending recursion.

I now introduced a load_cube() method that does the proper loading and that does not call set_mc_cone() again. This method is now used in fetch_cube().

…nvironment variable is not set.

Added a logger() method that allows setting the logger for an optimizer (and that also allows to set the NULL pointer)

To get the gradient we have to *divide* the gradient factor by the scale factor instead of multiplying it. This has been corrected now.

The flux() method now computes an approximated flux value for HealPix projections. This flux estimate is accurate to 1-3% level, dependent on pixel and Nside parameter.

The following methods have been added:
- set_z_phi
- loc2dir
- xyf2loc
- pix2xyf
- max_pixrad
- boundaries
- neighbours

The flux() methods return the correct flux in a pixel considering that a bilinear interpolation is performed in the map. Flux is defined as intensity times solid angle.

This works so far only properly for WCS projections. For HealPix maps, only the pixel intensity times the solid angle is returned.

The samp.hub.shutdown request allows a client to shutdown the VO Hub. This is not part of the SAMP protocol but does not harm. It is basically needed for unit testing.

The Hub is searched for when a client is initialised so that one can ping a Hub from a client.

The introduction of a timeout should avoid locking up the process.

Rename the methods and do some refactoring and clean up of the code.

Conflicts:
	src/vo/GVOClient.cpp
	test/test_GVO.cpp
	test/test_GVO.hpp

…lients connected ti the gammalib hub