isort, black, shorten long lines, apply->eval

Author: rocky

mathics/builtin/attributes.py

@@ -14,17 +14,17 @@
 """
 
 
-from mathics.builtin.base import Predefined, Builtin
+from mathics.builtin.base import Builtin, Predefined
 from mathics.core.assignment import get_symbol_list
 from mathics.core.atoms import String
 from mathics.core.attributes import (
-    attributes_bitset_to_list,
-    attribute_string_to_number,
     A_HOLD_ALL,
     A_HOLD_FIRST,
     A_LISTABLE,
     A_LOCKED,
     A_PROTECTED,
+    attribute_string_to_number,
+    attributes_bitset_to_list,
 )
 from mathics.core.expression import Expression
 from mathics.core.list import ListExpression
@@ -256,8 +256,8 @@ class HoldAllComplete(Predefined):
     <dl>
       <dt>'HoldAllComplete'
       <dd>is an attribute that includes the effects of 'HoldAll' and \
-        'SequenceHold', and also protects the function from being \
-        affected by the upvalues of any arguments.
+         'SequenceHold', and also protects the function from being \
+          affected by the upvalues of any arguments.
     </dl>
 
     'HoldAllComplete' even prevents upvalues from being used, and \
@@ -271,7 +271,8 @@ class HoldAllComplete(Predefined):
      = f[Sequence[a, b]]
     """
 
-    summary_text = "attribute for symbols that keep unevaluated all their elements, and discards upvalues"
+    summary_text = "attribute for symbols that keep unevaluated all \
+                    their elements, and discards upvalues"
 
 
 class HoldFirst(Predefined):
@@ -290,7 +291,8 @@ class HoldFirst(Predefined):
      = {HoldFirst, Protected, SequenceHold}
     """
 
-    summary_text = "attribute for symbols that keep unevaluated their first element"
+    summary_text = "attribute for symbols that keep unevaluated their \
+                    first element"
 
 
 class HoldRest(Predefined):

mathics/builtin/binary/io.py

@@ -4,37 +4,33 @@
 """
 
 import math
-import mpmath
 import struct
-import sympy
-
 from itertools import chain
 
+import mpmath
+import sympy
+
 from mathics.builtin.base import Builtin
 from mathics.core.atoms import Complex, Integer, MachineReal, Real, String
 from mathics.core.convert.expression import to_expression, to_mathics_list
 from mathics.core.convert.mpmath import from_mpmath
 from mathics.core.expression import Expression
-
-from mathics.core.systemsymbols import (
-    SymbolDirectedInfinity,
-    SymbolIndeterminate,
-)
-
 from mathics.core.number import dps
 from mathics.core.read import SymbolEndOfFile
 from mathics.core.streams import stream_manager
 from mathics.core.symbols import Symbol
-from mathics.core.systemsymbols import SymbolComplex
-
+from mathics.core.systemsymbols import (
+    SymbolComplex,
+    SymbolDirectedInfinity,
+    SymbolIndeterminate,
+)
 from mathics.eval.nevaluator import eval_N
 
 SymbolBinaryWrite = Symbol("BinaryWrite")
 
 
 class _BinaryFormat:
     """
-
     Container for BinaryRead readers and BinaryWrite writers
     """
 
@@ -363,14 +359,18 @@ def _UnsignedInteger128_writer(s, x):
 
 class BinaryRead(Builtin):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/BinaryRead.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/BinaryRead.html</url>
 
     <dl>
-    <dt>'BinaryRead[$stream$]'
+      <dt>'BinaryRead[$stream$]'
       <dd>reads one byte from the stream as an integer from 0 to 255.
-    <dt>'BinaryRead[$stream$, $type$]'
+
+      <dt>'BinaryRead[$stream$, $type$]'
       <dd>reads one object of specified type from the stream.
-    <dt>'BinaryRead[$stream$, {$type1$, $type2$, ...}]'
+
+      <dt>'BinaryRead[$stream$, {$type1$, $type2$, ...}]'
       <dd>reads a sequence of objects of specified types.
     </dl>
 
@@ -606,11 +606,11 @@ class BinaryRead(Builtin):
         "bfmt": "The stream `1` has been opened with BinaryFormat -> False and cannot be used with binary data.",
     }
 
-    def apply_empty(self, name, n, evaluation):
+    def eval_empty(self, name, n, evaluation):
         "BinaryRead[InputStream[name_, n_Integer]]"
-        return self.apply(name, n, None, evaluation)
+        return self.eval(name, n, None, evaluation)
 
-    def apply(self, name, n, typ, evaluation):
+    def eval(self, name, n, typ, evaluation):
         "BinaryRead[InputStream[name_, n_Integer], typ_]"
 
         channel = to_expression("InputStream", name, n)
@@ -660,7 +660,9 @@ def apply(self, name, n, typ, evaluation):
 
 class BinaryWrite(Builtin):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/BinaryWrite.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/BinaryWrite.html</url>
 
     <dl>
       <dt>'BinaryWrite[$channel$, $b$]'
@@ -899,11 +901,11 @@ class BinaryWrite(Builtin):
 
     writers = _BinaryFormat.get_writers()
 
-    def apply_notype(self, name, n, b, evaluation):
+    def eval_notype(self, name, n, b, evaluation):
         "BinaryWrite[OutputStream[name_, n_], b_]"
-        return self.apply(name, n, b, None, evaluation)
+        return self.eval(name, n, b, None, evaluation)
 
-    def apply(self, name, n, b, typ, evaluation):
+    def eval(self, name, n, b, typ, evaluation):
         "BinaryWrite[OutputStream[name_, n_], b_, typ_]"
 
         channel = to_expression("OutputStream", name, n)

mathics/builtin/binary/system.py

@@ -11,12 +11,15 @@
 
 class ByteOrdering(Predefined):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/ByteOrdering.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/ByteOrdering.html</url>
 
     <dl>
       <dt>'ByteOrdering'
-      <dd> is an option for BinaryRead, BinaryWrite, and related functions that specifies what ordering
-    of bytes should be assumed for your computer system..
+      <dd> is an option for BinaryRead, BinaryWrite, and related functions \
+      that specifies what ordering of bytes should be assumed for your \
+      computer system..
     </dl>
 
     X> ByteOrdering

mathics/builtin/comparison.py

@@ -10,19 +10,12 @@
 # This tells documentation how to sort this module
 sort_order = "mathics.builtin.testing-expressions"
 
-from typing import Optional, Any
+from typing import Any, Optional
 
 import sympy
 
-
-from mathics.builtin.base import (
-    BinaryOperator,
-    Builtin,
-    SympyFunction,
-)
-
+from mathics.builtin.base import BinaryOperator, Builtin, SympyFunction
 from mathics.builtin.numbers.constants import mp_convert_constant
-
 from mathics.core.atoms import (
     COMPARE_PREC,
     Complex,
@@ -42,7 +35,6 @@
     A_PROTECTED,
 )
 from mathics.core.convert.expression import to_expression, to_numeric_args
-from mathics.eval.nevaluator import eval_N
 from mathics.core.expression import Expression
 from mathics.core.number import dps
 from mathics.core.symbols import Atom, Symbol, SymbolFalse, SymbolList, SymbolTrue
@@ -55,7 +47,7 @@
     SymbolMaxPrecision,
     SymbolSign,
 )
-
+from mathics.eval.nevaluator import eval_N
 from mathics.eval.numerify import numerify
 
 operators = {
@@ -201,7 +193,7 @@ def numerify_args(items, evaluation) -> list:
 class _ComparisonOperator(_InequalityOperator):
     "Compares arguments in a chain e.g. a < b < c compares a < b and b < c."
 
-    def apply(self, items, evaluation):
+    def eval(self, items, evaluation):
         "%(name)s[items___]"
         items_sequence = items.get_sequence()
         if len(items_sequence) <= 1:
@@ -340,7 +332,7 @@ def equal2(self, lhs: Any, rhs: Any, max_extra_prec=None) -> Optional[bool]:
                 return c
         return None
 
-    def apply(self, items, evaluation):
+    def eval(self, items, evaluation):
         "%(name)s[items___]"
         items_sequence = items.get_sequence()
         n = len(items_sequence)
@@ -351,7 +343,7 @@ def apply(self, items, evaluation):
             for arg in items_sequence
         ]
         if not all(val is SymbolTrue for val in is_exact_vals):
-            return self.apply_other(items, evaluation)
+            return self.eval_other(items, evaluation)
         args = self.numerify_args(items, evaluation)
         for x, y in self.get_pairs(args):
             c = do_cplx_equal(x, y)
@@ -361,7 +353,7 @@ def apply(self, items, evaluation):
                 return SymbolFalse
         return SymbolTrue
 
-    def apply_other(self, args, evaluation):
+    def eval_other(self, args, evaluation):
         "%(name)s[args___?(!ExactNumberQ[#]&)]"
 
         args = args.get_sequence()
@@ -388,7 +380,7 @@ class _MinMax(Builtin):
         A_FLAT | A_NUMERIC_FUNCTION | A_ONE_IDENTITY | A_ORDERLESS | A_PROTECTED
     )
 
-    def apply(self, items, evaluation):
+    def eval(self, items, evaluation):
         "%(name)s[items___]"
         if hasattr(items, "flatten_with_respect_to_head"):
             items = items.flatten_with_respect_to_head(SymbolList)
@@ -447,7 +439,9 @@ def pairs(elements):
 
 class BooleanQ(Builtin):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/BooleanQ.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/BooleanQ.html</url>
 
     <dl>
       <dt>'BooleanQ[$expr$]'
@@ -481,7 +475,9 @@ class BooleanQ(Builtin):
 
 class Equal(_EqualityOperator, _SympyComparison):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/Equal.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/Equal.html</url>
 
     <dl>
       <dt>'Equal[$x$, $y$]'
@@ -644,7 +640,9 @@ class Greater(_ComparisonOperator, _SympyComparison):
 
 class GreaterEqual(_ComparisonOperator, _SympyComparison):
     """
-    <url>:WMA link:https://reference.wolfram.com/language/ref/GreaterEqual.html</url>
+    <url>
+    :WMA link:
+    https://reference.wolfram.com/language/ref/GreaterEqual.html</url>
 
     <dl>
       <dt>'GreaterEqual[$x$, $y$]'
@@ -690,7 +688,7 @@ class Inequality(Builtin):
     }
     summary_text = "chain of inequalities"
 
-    def apply(self, items, evaluation):
+    def eval(self, items, evaluation):
         "Inequality[items___]"
 
         elements = numerify(items, evaluation).get_sequence()
@@ -995,7 +993,7 @@ class SameQ(_ComparisonOperator):
 
     summary_text = "literal symbolic identity"
 
-    def apply_list(self, items, evaluation):
+    def eval_list(self, items, evaluation):
         "%(name)s[items___]"
         items_sequence = items.get_sequence()
         if len(items_sequence) <= 1:
@@ -1145,7 +1143,7 @@ class UnsameQ(_ComparisonOperator):
 
     summary_text = "not literal symbolic identity"
 
-    def apply_list(self, items, evaluation):
+    def eval_list(self, items, evaluation):
         "%(name)s[items___]"
         items_sequence = items.get_sequence()
         if len(items_sequence) <= 1: