From c6d80715cb355a805e1a0ec462082d6777b535aa Mon Sep 17 00:00:00 2001 From: Madison May Date: Thu, 11 Jun 2015 19:36:55 -0400 Subject: [PATCH] REMOVE: remove sentiment_hq from text_apis by default --- CHANGES.txt | 1 + indicoio/__init__.py | 2 +- indicoio/utils/multi.py | 3 +++ tests/test_remote.py | 3 --- 4 files changed, 5 insertions(+), 4 deletions(-) diff --git a/CHANGES.txt b/CHANGES.txt index e39c937..f48e333 100644 --- a/CHANGES.txt +++ b/CHANGES.txt @@ -25,3 +25,4 @@ v0.5.3, Wed Apr 15 -- Added scipy to requirements, edited Readme to not break py v0.6.0, Thu May 29 -- Remove numpy / scipy dependency in favor of Pillow v0.7.0, Tue Jun 9 -- Added support for calling multiple APIs in a single function and accepting filenames as image API inputs v0.7.1 Thu Jun 11 -- High quality sentiment API for private beta, fix for multi API support +v0.7.2 Thu Jun 11 -- Remove sentiment_hq from text apis by default diff --git a/indicoio/__init__.py b/indicoio/__init__.py index b766713..ed19a5c 100644 --- a/indicoio/__init__.py +++ b/indicoio/__init__.py @@ -1,6 +1,6 @@ from functools import partial -Version, version, __version__, VERSION = ('0.7.1',) * 4 +Version, version, __version__, VERSION = ('0.7.2',) * 4 JSON_HEADERS = { 'Content-type': 'application/json', diff --git a/indicoio/utils/multi.py b/indicoio/utils/multi.py index 94a4c7d..220f569 100644 --- a/indicoio/utils/multi.py +++ b/indicoio/utils/multi.py @@ -11,6 +11,9 @@ AVAILABLE_APIS = { 'image': IMAGE_APIS } +# TODO: remove this line when sentiment_hq is released publicly +AVAILABLE_APIS['text'].remove('sentiment_hq') + def multi(data, datatype, apis, batch=False, **kwargs): """ Helper to make multi requests of different types. diff --git a/tests/test_remote.py b/tests/test_remote.py index 0e80b0b..ef66a96 100644 --- a/tests/test_remote.py +++ b/tests/test_remote.py @@ -13,9 +13,6 @@ from indicoio import sentiment_hq, batch_sentiment_hq from indicoio import predict_image, predict_text, batch_predict_image, batch_predict_text from indicoio.utils.errors import IndicoError -# TODO: remove once sentiment_hq is added to the public API -config.TEXT_APIS.remove("sentiment_hq") - DIR = os.path.dirname(os.path.realpath(__file__)) class BatchAPIRun(unittest.TestCase):