Adding checks for printing optional attributes in __str__

Author: Neeratyoy

openml/datasets/dataset.py

@@ -181,13 +181,15 @@ def __str__(self):
         fields = {"Name": self.name,
                   "Version": self.version,
                   "Format": self.format,
-                  "Upload Date": self.upload_date.replace('T', ' '),
                   "Licence": self.licence,
                   "Download URL": self.url,
-                  "OpenML URL": "{}d/{}".format(base_url, self.dataset_id),
                   "Data file": self.data_file,
                   "Pickle file": self.data_pickle_file,
                   "# of features": len(self.features)}
+        if self.upload_date is not None:
+            fields["Upload Date"] = self.upload_date.replace('T', ' ')
+        if self.dataset_id is not None:
+            fields["OpenML URL"] = "{}d/{}".format(base_url, self.dataset_id)
         if self.qualities['NumberOfInstances'] is not None:
             fields["# of instances"] = int(self.qualities['NumberOfInstances'])
 

openml/evaluations/evaluation.py

@@ -1,5 +1,4 @@
 import openml.config
-import pandas as pd
 
 
 class OpenMLEvaluation(object):

openml/flows/flow.py

@@ -8,7 +8,6 @@
 from ..utils import extract_xml_tags, _tag_entity
 
 import openml.config
-import pandas as pd
 
 
 class OpenMLFlow(object):
@@ -140,12 +139,17 @@ def __str__(self):
         header = '{}\n{}\n'.format(header, '=' * len(header))
 
         base_url = "{}".format(openml.config.server[:-len('api/v1/xml')])
-        fields = {"Flow ID": "{} (version {})".format(self.flow_id, self.version),
-                  "Flow URL": "{}f/{}".format(base_url, self.flow_id),
-                  "Flow Name": self.name,
+        fields = {"Flow Name": self.name,
                   "Flow Description": self.description,
-                  "Upload Date": self.upload_date.replace('T', ' '),
                   "Dependencies": self.dependencies}
+        if self.flow_id is not None:
+            if self.version is not None:
+                fields["Flow ID"] = "{} (version {})".format(self.flow_id, self.version)
+            else:
+                fields["Flow ID"] = self.flow_id
+            fields["Flow URL"] = "{}f/{}".format(base_url, self.flow_id)
+        if self.upload_date is not None:
+            fields["Upload Date"] = self.upload_date.replace('T', ' ')
         if self.binary_url is not None:
             fields["Binary URL"] = self.binary_url
 

openml/runs/run.py

@@ -7,7 +7,6 @@
 import arff
 import numpy as np
 import xmltodict
-import pandas as pd
 
 import openml
 import openml._api_calls
@@ -70,20 +69,22 @@ def __str__(self):
 
         base_url = "{}".format(openml.config.server[:-len('api/v1/xml')])
         fields = {"Uploader Name": self.uploader_name,
-                  "Uploader Profile": "{}u/{}".format(base_url, self.uploader),
                   "Metric": self.task_evaluation_measure,
                   "Run ID": self.run_id,
-                  "Run URL": "{}r/{}".format(base_url, self.run_id),
                   "Task ID": self.task_id,
                   "Task Type": self.task_type,
-                  "Task URL": "{}t/{}".format(base_url, self.run_id),
+                  "Task URL": "{}t/{}".format(base_url, self.task_id),
                   "Flow ID": self.flow_id,
                   "Flow Name": self.flow_name,
                   "Flow URL": "{}f/{}".format(base_url, self.flow_id),
                   "Setup ID": self.setup_id,
                   "Setup String": self.setup_string,
                   "Dataset ID": self.dataset_id,
                   "Dataset URL": "{}d/{}".format(base_url, self.dataset_id)}
+        if self.uploader is not None:
+            fields["Uploader Profile"] = "{}u/{}".format(base_url, self.uploader)
+        if self.run_id is not None:
+            fields["Run URL"] = "{}r/{}".format(base_url, self.run_id)
         if self.task_evaluation_measure in self.evaluations:
             fields["Result"] = self.evaluations[self.task_evaluation_measure]
 

openml/setups/setup.py

@@ -1,5 +1,4 @@
 import openml.config
-import pandas as pd
 
 
 class OpenMLSetup(object):

openml/study/study.py

@@ -4,7 +4,6 @@
 import xmltodict
 
 import openml
-import pandas as pd
 
 
 class BaseStudy(object):
@@ -93,13 +92,16 @@ def __init__(
     def __str__(self):
         # header is provided by the sub classes
         base_url = "{}".format(openml.config.server[:-len('api/v1/xml')])
-        fields = {"ID": self.id,
-                  "Name": self.name,
+        fields = {"Name": self.name,
                   "Status": self.status,
-                  "Main Entity Type": self.main_entity_type,
-                  "Study URL": "{}s/{}".format(base_url, self.id),
-                  "Creator": "{}u/{}".format(base_url, self.creator),
-                  "Upload Time": self.creation_date.replace('T', ' ')}
+                  "Main Entity Type": self.main_entity_type}
+        if self.id is not None:
+            fields["ID"] = self.id
+            fields["Study URL"] = "{}s/{}".format(base_url, self.id)
+        if self.creator is not None:
+            fields["Creator"] = "{}u/{}".format(base_url, self.creator)
+        if self.creation_date is not None:
+            fields["Upload Time"] = self.creation_date.replace('T', ' ')
         if self.data is not None:
             fields["# of Data"] = len(self.data)
         if self.tasks is not None:

openml/tasks/task.py

@@ -47,9 +47,10 @@ def __str__(self):
         header = '{}\n{}\n'.format(header, '=' * len(header))
 
         base_url = "{}".format(openml.config.server[:-len('api/v1/xml')])
-        fields = {"Task Type": self.task_type,
-                  "Task ID": self.task_id,
-                  "Task URL": "{}t/{}".format(base_url, self.task_id)}
+        fields = {"Task Type": self.task_type}
+        if self.task_id is not None:
+            fields["Task ID"] = self.task_id
+            fields["Task URL"] = "{}t/{}".format(base_url, self.task_id)
         if self.evaluation_measure is not None:
             fields["Evaluation Measure"] = self.evaluation_measure
         if self.estimation_procedure is not None: