Tuning fix

[nhvaughan]

Changed the way tuning is configured. This loses functionality to switch doing tuning on and off on individual variables, but this feature was never used and deprecated long ago. Below are the changes I made. Don't need to read all of them, mostly here just for the record.

Main config file (see make_hists/Test_v14.json):

• Changed to version 14
• Deprecated the value useTuned, but code still recognize this and set accordingly.
• Added the value tunedmc, which can be set to "tuned", "untuned", or "both". This consolidates the naming convention (as previously there were multiple).
• Kept the value scalefileIn. Code now smart enough to turn tuning off (i.e. tunedmc set to "untuned") if this value is missing in config or left as a blank string.

weight_MCreScale:

• Removed bool m_useTuned member variable with std::string m_tunedmc that matches main config convention
• Recognizes new values from config, i.e. using the tunedmc value instead of useTuned, but should be smart enough to figure out what to do if you use useTuned.
• Smarter about defaulting to "untuned" in general
• When missing scalefileIn value in main config, defaults to no scale file (instead of a hardcoded file like before) and turns internal m_tunedmc to "untuned", turning tuning off entirely
  • may cause errors, or return negative scale factors, which the Fillers code should pick up and skip tuning

VariableFromConfig.h:

• Deprecated bool m_useTuned, replacing with std::string m_tunedmc following same main config convention
• Added std::string GetTunedMC and void SetTunedMC(std::string tunedmc) methods. The latter checks if tunedmc is in the right format
• Changed check values to turn tuning on and off in various methods to check m_tunedmc is set to "tuned", "untuned", or "both"
• Made InitializeTunedMCHistograms smarter so it can recognize if tuning is turned on (cleans up runsamplesMain.C a little)
• Cleaned up junk that hasn't been used in a long time (if ever)
• Fixed to match Google C++ formatting (thanks VSCode for making that easy!)

Variable2DFromConfig.h:

• Much the same as VariableFromConfig (switch to m_tunedmc, checks on that string value, clean up and Google C++ formatting)

GetVariablesFromConfig:

• Added std::string tunedmc argument that defaults to "untuned".
• Use new method VariableFromConfig::SetTunedMC to set tunedmc value after initialization

GetVariables2DFromConfig:

• Cleanup

Fillers.h:

• Changed check value for filling tuned events to use new tunedmc convention.
  • Would have done an enum here, but for the sake of readibility left it this way.

runsamplesMain.C:

• Added check for tunedmc in config (defaults to "untuned").
• Still checks for useTuned for backwards compatibility on old configs, but it'll complain about it.
• Cleaned up commented out junk from previous tuning switch schemes
• Added various changes to accommodate above changes
  • GetVariables new argument for tunedmc
  • Running InitializeTunedMCHistograms without a check on tunedmc (VariableFromConfig is smart enough to handle this on its own now)
• Added cout at the end with the outfile name so it's easier to access to check after running.

[nhvaughan]

You can check out this pull request to see how it works. There are no conflicts with the current state of main, and if you're not doing anything in any of the files changed here there shouldn't be any other conflicts.