Embedders - temporarily fix get_proxy function

orangecontrib/text/__init__.py

@@ -1,5 +1,42 @@
 # Set where NLTK data is downloaded
 import os
+
+# temporary solution - remove when Orange 3.33 is released
+# it must be imported before nltk_data_dir
+from typing import Optional, Dict
+from Orange.misc.utils import embedder_utils
+
+
+def _get_proxies() -> Optional[Dict[str, str]]:
+    """
+    Return dict with proxy addresses if they exist.
+    Returns
+    -------
+    proxy_dict
+        Dictionary with format {proxy type: proxy address} or None if
+        they not set.
+    """
+    def add_scheme(url: Optional[str]) -> Optional[str]:
+        if url is not None and "://" not in url:
+            # if no scheme default to http - as other libraries do (e.g. requests)
+            return f"http://{url}"
+        else:
+            return url
+
+    http_proxy = add_scheme(os.environ.get("http_proxy"))
+    https_proxy = add_scheme(os.environ.get("https_proxy"))
+    proxy_dict = {}
+    if http_proxy:
+        proxy_dict["http://"] = http_proxy
+    if https_proxy:
+        proxy_dict["https://"] = https_proxy
+    return proxy_dict if proxy_dict else None
+
+
+embedder_utils.get_proxies = _get_proxies
+# remove to here
+
+
 from orangecontrib.text.misc import nltk_data_dir
 os.environ['NLTK_DATA'] = nltk_data_dir()
 

orangecontrib/text/tests/test_documentembedder.py

@@ -144,6 +144,16 @@ def test_invalid_parameters(self):
         with self.assertRaises(ValueError):
             self.embedder = DocumentEmbedder(aggregator='average')
 
+    def test_remove_temporary_proxy_solution(self):
+        """
+        When it starts to fail:
+        - remove this test
+        - remove temporary implementation of get_proxy() function in text.__inint__
+        - set minimum version of Orange on 3.33
+        """
+        import Orange
+        self.assertGreater("3.33.0", Orange.__version__)
+
 
 if __name__ == "__main__":
     unittest.main()