noqa: E501 tidying

Author: RudolfCardinal

cardinal_pythonlib/argparse_func.py

@@ -55,7 +55,7 @@ class ShowAllSubparserHelpAction(_HelpAction):
     shows help for all subparsers. As per
 
     https://stackoverflow.com/questions/20094215/argparse-subparser-monolithic-help-output
-    """  # noqa
+    """  # noqa: E501
 
     def __call__(
         self,
@@ -147,7 +147,7 @@ def str2bool(v: str) -> bool:
             default=NICE,  # if the argument is entirely absent
             help="Activate nice mode.")
 
-    """  # noqa
+    """  # noqa: E501
     lv = v.lower()
     if lv in ("yes", "true", "t", "y", "1"):
         return True

cardinal_pythonlib/athena_ohdsi.py

@@ -213,7 +213,7 @@ def match_tuple(self, target: "AthenaConceptRow") -> Tuple[float, float]:
         - https://www.nltk.org/
 
         BINNED. We will use human judgement.
-        """  # noqa
+        """  # noqa: E501
         self_words = set(x.lower() for x in self.concept_name.split())
         other_words = set(x.lower() for x in target.concept_name.split())
         # More matching words better

cardinal_pythonlib/buildfunc.py

@@ -136,7 +136,7 @@ def git_clone(
         )
         run_func([git, "-C", directory, "reset", "--hard", commit])
         # Using a Git repository that's not in the working directory:
-        # https://stackoverflow.com/questions/1386291/git-git-dir-not-working-as-expected  # noqa
+        # https://stackoverflow.com/questions/1386291/git-git-dir-not-working-as-expected  # noqa: E501
     return True
 
 

cardinal_pythonlib/classes.py

@@ -37,7 +37,7 @@
 https://stackoverflow.com/questions/1776994
 https://docs.python.org/3/library/inspect.html
 https://github.com/edoburu/django-fluent-contents/issues/43
-https://bytes.com/topic/python/answers/843424-python-2-6-3-0-determining-if-method-inherited  # noqa
+https://bytes.com/topic/python/answers/843424-python-2-6-3-0-determining-if-method-inherited  # noqa: E501
 https://docs.python.org/3/reference/datamodel.html
 
 In Python 2, you can do this:

cardinal_pythonlib/cmdline.py

@@ -48,14 +48,14 @@ def cmdline_split(s: str, platform: Union[int, str] = "this") -> List[str]:
             - ``1`` = POSIX;
             - ``0`` = Windows/CMD
             - (other values reserved)
-    """  # noqa
+    """  # noqa: E501
     if platform == "this":
         platform = sys.platform != "win32"  # RNC: includes 64-bit Windows
 
     if platform == 1:  # POSIX
-        re_cmd_lex = r""""((?:\\["\\]|[^"])*)"|'([^']*)'|(\\.)|(&&?|\|\|?|\d?\>|[<])|([^\s'"\\&|<>]+)|(\s+)|(.)"""  # noqa
+        re_cmd_lex = r""""((?:\\["\\]|[^"])*)"|'([^']*)'|(\\.)|(&&?|\|\|?|\d?\>|[<])|([^\s'"\\&|<>]+)|(\s+)|(.)"""  # noqa: E501
     elif platform == 0:  # Windows/CMD
-        re_cmd_lex = r""""((?:""|\\["\\]|[^"])*)"?()|(\\\\(?=\\*")|\\")|(&&?|\|\|?|\d?>|[<])|([^\s"&|<>]+)|(\s+)|(.)"""  # noqa
+        re_cmd_lex = r""""((?:""|\\["\\]|[^"])*)"?()|(\\\\(?=\\*")|\\")|(&&?|\|\|?|\d?>|[<])|([^\s"&|<>]+)|(\s+)|(.)"""  # noqa: E501
     else:
         raise AssertionError(f"unknown platform {platform!r}")
 

cardinal_pythonlib/compression.py

@@ -88,7 +88,7 @@ def gzip_string(text: str, encoding: str = "utf-8") -> bytes:
         print(gz1 == gz2)  # False
         # ... but the difference is probably in the timestamp bytes!
 
-    """  # noqa
+    """  # noqa: E501
     data = text.encode(encoding)
     return gzip.compress(data)
 

cardinal_pythonlib/contexts.py

@@ -38,5 +38,5 @@ def dummy_context_mgr():
     - https://stackoverflow.com/questions/27803059/conditional-with-statement-in-python
     - See also
       https://stackoverflow.com/questions/893333/multiple-variables-in-a-with-statement
-    """  # noqa
+    """  # noqa: E501
     yield None

cardinal_pythonlib/datetimefunc.py

@@ -222,7 +222,7 @@ def pendulum_to_utc_datetime_without_tz(x: DateTime) -> datetime.datetime:
         dt_utc_from_moscow = pendulum_to_utc_datetime_without_tz(in_moscow)  # 6am, no timezone info
         dt_utc_from_london = pendulum_to_utc_datetime_without_tz(in_london)  # 6am, no timezone info
 
-    """  # noqa
+    """  # noqa: E501
     pendulum_in_utc = pendulum.UTC.convert(x)
     return pendulum_to_datetime_stripping_tz(pendulum_in_utc)
 

cardinal_pythonlib/dbfunc.py

@@ -56,7 +56,7 @@ def genrows(
     Yields:
         each row
     """
-    # http://code.activestate.com/recipes/137270-use-generators-for-fetching-large-db-record-sets/  # noqa
+    # http://code.activestate.com/recipes/137270-use-generators-for-fetching-large-db-record-sets/  # noqa: E501
     while True:
         results = cursor.fetchmany(arraysize)
         if not results:

cardinal_pythonlib/debugging.py

@@ -33,10 +33,14 @@
 import sys
 import traceback
 from types import FrameType
-from typing import Any, Callable, List, Optional
+from typing import Any, Callable, List, Optional, TYPE_CHECKING
 
 from cardinal_pythonlib.logs import get_brace_style_log_with_null_handler
 
+if TYPE_CHECKING:
+    # noinspection PyUnresolvedReferences
+    from inspect import FrameInfo
+
 log = get_brace_style_log_with_null_handler(__name__)
 
 
@@ -80,7 +84,7 @@ def get_class_name_from_frame(fr: FrameType) -> Optional[str]:
     If the call was to a member function of a class, this function attempts
     to read the class's name. It returns ``None`` otherwise.
     """
-    # https://stackoverflow.com/questions/2203424/python-how-to-retrieve-class-information-from-a-frame-object  # noqa
+    # https://stackoverflow.com/questions/2203424/python-how-to-retrieve-class-information-from-a-frame-object  # noqa: E501
     args, _, _, value_dict = inspect.getargvalues(fr)
     # we check the first parameter for the frame function is named 'self'
     if len(args) and args[0] == "self":
@@ -138,7 +142,7 @@ def f1():
         That was called by: f1
 
     """
-    # https://stackoverflow.com/questions/5067604/determine-function-name-from-within-that-function-without-using-traceback  # noqa
+    # https://stackoverflow.com/questions/5067604/determine-function-name-from-within-that-function-without-using-traceback  # noqa: E501
     try:
         # noinspection PyProtectedMember
         frame = sys._getframe(back + 2)
@@ -241,7 +245,7 @@ def f1():
     # "0 back" is debug_callers, so "1 back" its caller
     # https://docs.python.org/3/library/inspect.html
     callers = []  # type: List[str]
-    frameinfolist = inspect.stack()  # type: List[FrameInfo]  # noqa
+    frameinfolist = inspect.stack()  # type: List[FrameInfo]
     frameinfolist = frameinfolist[start_back:]
     for frameinfo in frameinfolist:
         frame = frameinfo.frame