Improve comments

Author: mfeurer

openml/flows/flow.py

@@ -355,6 +355,8 @@ def _add_if_nonempty(dic, key, value):
 
 
 def _check_flow(flow):
+    # Import is not possible at the top of the file as this would cause an
+    # ImportError due to an import cycle.
     import openml.flows.functions
 
     flow_copy = openml.flows.functions.get_flow(flow.flow_id)

openml/flows/functions.py

@@ -144,8 +144,6 @@ def assert_flows_equal(flow1, flow2):
         raise TypeError('Argument 2 must be of type OpenMLFlow, but is %s' %
                         type(flow2))
 
-    # Name is actually not generated by the server, but it will be
-    # tested further down with a getter (allows mocking in the tests)
     generated_by_the_server = ['flow_id', 'uploader', 'version',
                                'upload_date', ]
     ignored_by_python_API = ['binary_url', 'binary_format', 'binary_md5',
@@ -170,4 +168,4 @@ def assert_flows_equal(flow1, flow2):
             if attr1 != attr2:
                 raise ValueError("Flow %s: values for attribute '%s' differ: "
                                  "'%s' vs '%s'." %
-                                 (str(flow1.name), str(key), str(attr1), str(attr2)))
+                                 (str(flow1.name), str(key), str(attr1), str(attr2)))