some flake8 for repl

src/sage/repl/attach.py

@@ -84,6 +84,7 @@
 load_debug_mode = False
 attach_debug_mode = True
 
+
 def load_attach_mode(load_debug=None, attach_debug=None):
     """
     Get or modify the current debug mode for the behavior of
@@ -131,14 +132,15 @@ def load_attach_mode(load_debug=None, attach_debug=None):
     global load_debug_mode, attach_debug_mode
     if load_debug is None and attach_debug is None:
         return (load_debug_mode, attach_debug_mode)
-    if not load_debug is None:
+    if load_debug is not None:
         load_debug_mode = load_debug
-    if not attach_debug is None:
+    if attach_debug is not None:
         attach_debug_mode = attach_debug
 
 
 search_paths = []
 
+
 def load_attach_path(path=None, replace=False):
     """
     Get or modify the current search path for :func:`load` and
@@ -169,7 +171,7 @@ def load_attach_path(path=None, replace=False):
         sage: with open(fullpath, 'w') as f:
         ....:     _ = f.write("print(37 * 3)")
 
-    We put SAGE_TMP on the attach path for testing (otherwise this will
+    We put ``SAGE_TMP`` on the attach path for testing (otherwise this will
     load ``test.py`` from the current working directory if that happens
     to exist)::
 
@@ -226,8 +228,7 @@ def load_attach_path(path=None, replace=False):
 
 def reset_load_attach_path():
     """
-    Resets the current search path for :func:`load` and
-    :func:`attach`.
+    Reset the current search path for :func:`load` and :func:`attach`.
 
     The default path is ``'.'`` plus any paths specified in the
     environment variable ``SAGE_LOAD_ATTACH_PATH``.
@@ -266,6 +267,7 @@ def reset_load_attach_path():
     for path in os.environ.get('SAGE_LOAD_ATTACH_PATH', '').split(':'):
         load_attach_path(path=path)
 
+
 # Set up the initial search path for loading and attaching files.  A
 # user can modify the path with the function load_attach_path.
 reset_load_attach_path()
@@ -310,7 +312,7 @@ def attach(*files):
     .. SEEALSO::
 
         :meth:`~sage.repl.load.load` is the same as :func:`attach`, but
-        doesn't automatically reload a file when it changes.
+        does not automatically reload a file when it changes.
 
     EXAMPLES:
 
@@ -487,6 +489,7 @@ def detach(filename):
     if not attached:
         sage.repl.inputhook.uninstall()
 
+
 def reset():
     """
     Remove all the attached files from the list of attached files.

src/sage/repl/display/fancy_repr.py

@@ -2,7 +2,6 @@
 """
 Representations of objects
 """
-
 # ****************************************************************************
 #       Copyright (C) 2014 Volker Braun <vbraun.name@gmail.com>
 #
@@ -20,10 +19,10 @@
     _type_pprinters,
 )
 
-_baseclass_reprs = (object.__repr__,)
-
 from sage.repl.display.util import format_list
 
+_baseclass_reprs = (object.__repr__,)
+
 
 class ObjectReprABC(object):
     """
@@ -216,7 +215,6 @@ def __call__(self, obj, p, cycle):
         return True
 
 
-
 class PlainPythonRepr(ObjectReprABC):
     """
     The ordinary Python representation
@@ -360,5 +358,3 @@ def __call__(self, obj, p, cycle):
             return False
         p.text(output)
         return True
-
-

src/sage/repl/display/formatter.py

@@ -51,14 +51,14 @@
 :class:`~sage.typeset.ascii_art.AsciiArt` object (see and
 :meth:`sage.structure.sage_object.SageObject._ascii_art_`).  """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2014 Volker Braun <vbraun.name@gmail.com>
 #
 #  Distributed under the terms of the GNU General Public License (GPL)
 #  as published by the Free Software Foundation; either version 2 of
 #  the License, or (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 from io import StringIO
 
@@ -290,4 +290,3 @@ def __call__(self, obj):
         printer.pretty(obj)
         printer.flush()
         return stream.getvalue()
-

src/sage/repl/display/jsmol_iframe.py

@@ -28,8 +28,7 @@
 from sage.misc.cachefunc import cached_method
 
 
-INNER_HTML_TEMPLATE = \
-"""
+INNER_HTML_TEMPLATE = """
 <html>
 <head>
   <style>
@@ -38,7 +37,7 @@
       padding: 0;
       overflow: hidden;
     }}
-    body, html {{      
+    body, html {{
       height: 100%;
       width: 100%;
     }}
@@ -67,18 +66,16 @@
 """
 
 
-IFRAME_TEMPLATE = \
-"""
-<iframe srcdoc="{escaped_inner_html}" 
+IFRAME_TEMPLATE = """
+<iframe srcdoc="{escaped_inner_html}"
         width="{width}"
         height="{height}"
         style="border: 0;">
 </iframe>
 """
 
 
-OUTER_HTML_TEMPLATE = \
-"""
+OUTER_HTML_TEMPLATE = """
 <html>
 <head>
   <title>JSmol 3D Scene</title>
@@ -102,7 +99,7 @@ def __init__(self, jmol, path_to_jsmol=None, width='100%', height='100%'):
 
         - ``path_to_jsmol`` -- string (optional, default is
           ``'/nbextensions/jsmol'``). The path (relative or absolute)
-          where ``JSmol.min.js`` is served on the web server. 
+          where ``JSmol.min.js`` is served on the web server.
 
         - ``width`` -- integer or string (optional, default:
           ``'100%'``). The width of the JSmol applet using CSS
@@ -176,7 +173,7 @@ def js_script(self):
         Since the many shortcomings of Javascript include multi-line
         strings, this actually returns Javascript code to reassemble
         the script from a list of strings.
-        
+
         OUTPUT:
 
         String. Javascript code that evaluates to :meth:`script` as
@@ -199,13 +196,13 @@ def js_script(self):
             script += [r"  '{0}',".format(line)]
         script += [r"].join('\n');"]
         return '\n'.join(script)
-        
+
     def _repr_(self):
         """
         Return as string representation
 
         OUTPUT:
-        
+
         String.
 
         EXAMPLES::
@@ -242,15 +239,15 @@ def inner_html(self):
             height=self._height,
             path_to_jsmol=self._path,
         )
-        
+
     def iframe(self):
         """
         Return HTML iframe
 
         OUTPUT:
-        
+
         String.
-        
+
         EXAMPLES::
 
             sage: from sage.repl.display.jsmol_iframe import JSMolHtml

src/sage/repl/display/pretty_print.py

@@ -14,21 +14,21 @@
 - Volker Braun (2013): refactored into DisplayHookBase
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2014 Volker Braun <vbraun.name@gmail.com>
 #
 #  Distributed under the terms of the GNU General Public License (GPL)
 #  as published by the Free Software Foundation; either version 2 of
 #  the License, or (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 
 from IPython.lib.pretty import PrettyPrinter
 
-from sage.repl.display.fancy_repr import *
-
-
+from sage.repl.display.fancy_repr import (TallListRepr, PlainPythonRepr,
+                                          LargeMatrixHelpRepr,
+                                          SomeIPythonRepr)
 
 
 class SagePrettyPrinter(PrettyPrinter):
@@ -139,9 +139,11 @@ def pretty(self, obj):
         try:
             ok = False
             for representation in self.pretty_repr:
-                if self.DEBUG: print('Trying {0}'.format(representation))
+                if self.DEBUG:
+                    print('Trying {0}'.format(representation))
                 ok = representation(obj, self, cycle)
-                if self.DEBUG: print('ok = {0}'.format(ok))
+                if self.DEBUG:
+                    print('ok = {0}'.format(ok))
                 if ok not in [True, False]:
                     raise RuntimeError('printer failed to return boolean')
                 if ok:
@@ -151,4 +153,3 @@ def pretty(self, obj):
         finally:
             self.end_group()
             self.stack.pop()
-

src/sage/repl/image.py

@@ -23,14 +23,14 @@
     <class 'sage.repl.image.Image'>
 """
 
-#*****************************************************************************
+# ****************************************************************************
 #       Copyright (C) 2015 Volker Braun <vbraun.name@gmail.com>
 #
 #  Distributed under the terms of the GNU General Public License (GPL)
 #  as published by the Free Software Foundation; either version 2 of
 #  the License, or (at your option) any later version.
-#                  http://www.gnu.org/licenses/
-#*****************************************************************************
+#                  https://www.gnu.org/licenses/
+# ****************************************************************************
 
 import io
 
@@ -48,7 +48,7 @@ def __init__(self, mode, size, color='white'):
 
         - ``mode`` -- string. The mode to use for the new image. Valid
           options are:
-                     
+
               * ``'1'`` (1-bit pixels, black and white, stored with
                 one pixel per byte)
 
@@ -85,7 +85,7 @@ def __init__(self, mode, size, color='white'):
           initialised.
 
         OUTPUT:
-        
+
         A new :class:`Image` object.
 
         EXAMPLES::
@@ -100,7 +100,7 @@ def __init__(self, mode, size, color='white'):
     def pil(self):
         """
         Access the wrapped PIL(low) Image
-        
+
         OUTPUT:
 
         The underlying ``PIL.Image.Image object``.
@@ -131,7 +131,7 @@ def pixels(self):
             <PixelAccess object at 0x...>
         """
         return self._pil.load()
-    
+
     def _repr_(self):
         """
         Return string representation.
@@ -169,7 +169,7 @@ def _repr_(self):
     def mode(self):
         """
         Return the color mode
-        
+
         OUTPUT:
 
         String. As given when constructing the image.
@@ -201,7 +201,7 @@ def width(self):
             34
         """
         return self.pil.size[0]
-    
+
     def height(self):
         """
         Return the vertical dimension in pixels

src/sage/repl/inputhook.py

@@ -32,7 +32,7 @@ def sage_inputhook(context):
     while True:
         sage.repl.attach.reload_attached_files_if_modified()
         try:
-            r, w, e = select.select([f], [], [], TIMEOUT)
+            r, _, _ = select.select([f], [], [], TIMEOUT)
             if f in r:
                 return  # IPython signalled us to stop
         except select.error as e: