updated README and dependencies

Author: TjarkMiener

README.md

@@ -34,10 +34,11 @@ NOTE for developers: If you wish to fork/clone the respository and make changes
 
 ### Dependencies
 
-- Python 3.7.3
+- Python 3.X
 - NumPy
 - SciPy
 - Pandas
+- PyTables
 - PyYAML
 - Matplotlib
 
@@ -57,7 +58,7 @@ import yaml
 from likelihood_combiner.combiner import run_combiner
 
 with open('</installation/path>/config/config.yml', 'r') as myconfig:
-    config = yaml.load(myconfig)
+    config = yaml.safe_load(myconfig)
 run_combiner(config)
 ```
 

environment.yml

@@ -4,7 +4,7 @@ channels:
     - conda-forge
     - menpo
 dependencies:
-    - python=3.7.3
+    - python
     - matplotlib
     - numpy
     - scipy

likelihood_combiner/combiner.py

@@ -137,7 +137,7 @@ def run_combiner(config, sigmavULs, sigmavULs_Jnuisance, simulations=[-1]):
     args = parser.parse_args()
 
     with open(args.config_file, 'r') as config_file:
-        config = yaml.load(config_file)
+        config = yaml.safe_load(config_file)
 
     try:
         hdf5file = config['Data']['hdf5_dataset']

likelihood_combiner/utils.py

@@ -87,7 +87,7 @@ def plot_sigmavULs(hdf5file, output_dir, config):
         if config['Data']['J_nuisance']:
             sigmavULs_Jnuisance = pd.read_hdf(hdf5file, key='{}/sigmavULs_Jnuisance'.format(channel))
         for ul_dict in [sigmavULs,sigmavULs_Jnuisance]:
-            if not ul_dict:
+            if not dict(ul_dict):
                 continue 
             masses = np.squeeze(ul_dict[['masses']].to_numpy())
             data = np.squeeze(ul_dict[['data']].to_numpy())

setup.py

@@ -1,7 +1,7 @@
 from setuptools import setup, find_packages
 
 setup(name='lklcom',
-      version='0.2.0',
+      version='0.3.0',
       description='LikelihoodCombiner combines likelihoods from different experiments.',
       url='https://github.com/TjarkMiener/likelihood_combiner',
       license='BSD-3-Clause',