modified:   ../.gitignore
	modified:   ../containers/singularity/Singularity
	modified:   Scandalizer.cxx
	modified:   THcParmList.cxx
	new file:   ../tests/README.md

- removed spdlog ( now in analyzer)
- added logging to THaAnalzyer (podd)

This includes a particularly nasty leak in the shower code that caused
several hundred bytes to be lost _per event_.

- Mixins defined in Logger.h
- Mixin used in base classes (THcDC is currently a good example)

Added a few more spdlog loggers

Added logging to many.

- Added logging to hodoscope, aerogel hitlist and detectormap
- Working on scandalizer features.

Added logging.

Added more logging.

added more logging.

More logging.

  Previously, if load was called with a run number, array continuation
  lines in included files were being interpreted as lists of run numbers.
  Still we have the limitation that in the top file, if we define an array
  with multiple lines, the continuation lines will be interpreted as
  run number lists and mess things up.  But it is unlikely that we would
  need to define arrays in the top file when we are in run number
  mode.

  Add doxygen groups "Apparatuses" and "Decoders"
  Make sure every class is in a group
  Make sure most classes have a \brief description
  Improve comments for some classes including THcHallCSpectrometer
     and hodoscope classes
  Change some Doxyfile defaults.  Include sources files.

	modified:   CMakeLists.txt
	modified:   THcParmList.cxx
	renamed:    json.hpp -> nlohmann/json.hpp

	modified:   THcParmList.cxx
	modified:   THcParmList.h
	new file:   json.hpp

  When Load is called with a run number, then the input file is ignored
  until a line with a list of run numbers or run number ranges is found.
  For example:
     44,50-90,113
  If the run number matches one of these run numbers or ranges,
  parameter definitions after the match are loaded.  (Until the next
  list of run numbers and ranges.)
  Before this commit, only a single run number or a single range of
  run numbers was allowed.

  If Load is called with a run number, but the first non blank/comment
  line encountered is not a run number or range, print a warning message.

THcParmList::LoadParmValues now throws runtime_error instead of calling Fatal,
and with that causing core dump, when non-optional parameter is not found.

  Modify THcFormula and THcParmList to match changes to THaFormula
     (needed for reports)
  Small changes in THcInterface and detector classes for compatibility
     with new OO decoder and podd 1.6

  For the most part this is making sure that the loop variable in
  a for loop is of the same type as the variable that it is compared
  against (Often a UInt_t).

This reverts commit e9ce1040.

Conflicts:
	src/THcHallCSpectrometer.cxx

  For the most part this is making sure that the loop variable in
  a for loop is of the same type as the variable that it is compared
  against (Often a UInt_t).

  Don't put [1] at end of variable name as is done with
  numerical parameters with Define.

  Use new CCDB methods to get table row and column counts

  Title now copied to THcParmList variable
  String variables copied.  (Only first element of string array copied)
  Parameter created as int or double depending on CCDB table type
  Multiple column tables ignored.

  CCDB must be installed and CCDB_HOME defined before compiling.
  If CCDB_HOME is not defined, hcana is compiled without CCDB support
  scons has not been updated to compile with CCDB support
  To use, instead of gHcParms->Load(filename), do
    gHcParms->OpenCCDB(runnum, ccdb_connection_string)
    gHcParms->LoadCCDBDirectory("hms","h")
    gHcParms->LoadCCDBDirectory("sos","s")
    gHcParms->LoadCCDBDirectory("gen","g")
  Parmaters will be loaded into the same internal gHcParms list as with
  ThcParmList::Load, so no changes to detector classes are needed.
  Scripts to convert existing parameters into CCDB are under development.
  Currently all ccdb constants are loaded as doubles as there is not
  a way to easily query ccdb to get the data type.
  Also strings are not loaded.

These were related to a) scope of some variable declarations, and b) use
of pre-increment vs. post-increment of non-primitive types (this has a
slight performance hit due to the way that pre- vs. post-increment
operators are overloaded in C++.

Also cleaned up a few places where initialized variables were not being
used.

  Use HALLC_MODS in make file to turn on Hall C changes in Podd.
  Remove unneeded and now broken includes of THaDB.h.

Use DBRequest inside of loops over planes, using Form
to construct variables names that have the plane lable in them
(e.g. 1x, 1y, 2x, 2y)

prepended to all variables in the DBRequest list.