FIX #192 use https when possible

Author: mfeurer

openml/__init__.py

@@ -1,7 +1,7 @@
 """
 The OpenML module implements a python interface to
-`OpenML <http://openml.org>`_, a collaborative platform for machine learning.
-OpenML can be used to
+`OpenML <https://www.openml.org>`_, a collaborative platform for machine
+learning. OpenML can be used to
 
 * store, download and analyze datasets
 * make experiments and their results (e.g. models, predictions)
@@ -10,7 +10,7 @@
   meta studies
 
 In particular, this module implemts a python interface for the
-`OpenML REST API <http://openml.org/guide#!rest_services>`_
+`OpenML REST API <https://www.openml.org/guide#!rest_services>`_
 (`REST on wikipedia
 <http://en.wikipedia.org/wiki/Representational_state_transfer>`_).
 """

openml/config.py

@@ -11,7 +11,7 @@
     format='[%(levelname)s] [%(asctime)s:%(name)s] %('
            'message)s', datefmt='%H:%M:%S')
 
-server = "http://www.openml.org/api/v1/xml"
+server = "https://www.openml.org/api/v1/xml"
 apikey = ""
 cachedir = ""
 privatedir = ""

openml/datasets/functions.py

@@ -175,7 +175,7 @@ def _list_datasets(api_call):
     assert type(datasets_dict['oml:data']['oml:dataset']) == list, \
         type(datasets_dict['oml:data'])
     assert datasets_dict['oml:data']['@xmlns:oml'] == \
-           'http://openml.org/openml'
+           'http://openml.org/openml', datasets_dict['oml:data']['@xmlns:oml']
 
     datasets = dict()
     for dataset_ in datasets_dict['oml:data']['oml:dataset']:

openml/flows/functions.py

@@ -20,7 +20,7 @@ def _check_flow_exists(name, version):
 
     Notes
     -----
-    see http://www.openml.org/api_docs/#!/flow/get_flow_exists_name_version
+    see https://www.openml.org/api_docs/#!/flow/get_flow_exists_name_version
     """
     if not (type(name) is str and len(name) > 0):
         raise ValueError('Argument \'name\' should be a non-empty string')

openml/runs/functions.py

@@ -264,7 +264,7 @@ def list_runs(offset=None, size=None, id=None, task=None,
               flow=None, uploader=None, tag=None):
     """List all runs matching all of the given filters.
 
-    Perform API call `/run/list/{filters} <http://www.openml.org/api_docs/#!/run/get_run_list_filters>`_
+    Perform API call `/run/list/{filters} <https://www.openml.org/api_docs/#!/run/get_run_list_filters>`_
 
     Parameters
     ----------

openml/tasks/functions.py

@@ -96,7 +96,7 @@ def list_tasks(task_type_id=None, offset=None, size=None, tag=None):
     ----------
     task_type_id : int, optional
         ID of the task type as detailed
-        `here <http://www.openml.org/search?type=task_type>`_.
+        `here <https://www.openml.org/search?type=task_type>`_.
     offset : int, optional
         the number of tasks to skip, starting from the first
     size : int, optional
@@ -131,6 +131,8 @@ def list_tasks(task_type_id=None, offset=None, size=None, tag=None):
 
 def _list_tasks(api_call):
     return_code, xml_string = _perform_api_call(api_call)
+    with open('/tmp/list_tasks.xml', 'w') as fh:
+        fh.write(xml_string)
     tasks_dict = xmltodict.parse(xml_string)
     # Minimalistic check if the XML is useful
     if 'oml:tasks' not in tasks_dict:

openml/testing.py

@@ -42,7 +42,8 @@ def setUp(self):
         self.cached = True
         # amueller's read/write key that he will throw away later
         openml.config.apikey = "610344db6388d9ba34f6db45a3cf71de"
-        openml.config.server = "http://test.openml.org/api/v1/xml"
+        #openml.config.server = "http://capa.win.tue.nl/api/v1/xml"
+        openml.config.server = "https://test.openml.org/api/v1/xml"
         openml.config.set_cache_directory(self.workdir, self.workdir)
 
     def tearDown(self):