Fixed problem with missing Redis

cuemacro · Dec 16, 2019 · dec6827 · dec6827
1 parent e574d82
commit dec6827
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -86,6 +86,8 @@ In findatapy/examples you will find several demos
 
 # Coding log
 
+* 16 Dec 2019
+    * Fix problem with missing Redis dependency when reading from market
 * 04 Dec 2019
     * Allow usage on Azure Notebooks, by making keyring dependency optional
 * 03 Nov 2019

diff --git a/findatapy/market/ioengine.py b/findatapy/market/ioengine.py
@@ -555,7 +555,7 @@ def read_time_series_cache_from_disk(self, fname, engine='hdf5', start_date=None
                 # for pyarrow
                 context = pa.default_serialization_context()
 
-                if True:
+                try:
                     r = redis.StrictRedis(host=db_server, port=db_port, db=0)
 
                     # is there a compressed key?
@@ -579,7 +579,7 @@ def read_time_series_cache_from_disk(self, fname, engine='hdf5', start_date=None
                             msg = context.deserialize(msg)
                             #self.logger.warning("Key " + fname_single + " not in Redis cache?")
 
-                else:
+                except:
                     self.logger.info("Cache not existent for " + fname_single + " in Redis")
 
                 if msg is None:

diff --git a/setup.py b/setup.py
@@ -5,7 +5,7 @@
 tickers, using configuration files. There is also functionality which is particularly useful for those downloading FX market data."""
 
 setup(name='findatapy',
-      version='0.1.4',
+      version='0.1.5',
       description='Market data library',
       author='Saeed Amen',
       author_email='saeed@cuemacro.com',