Use correct linter settings

Author: amas0

cmdstanpy/__init__.py

@@ -37,13 +37,13 @@ def _cleanup_tmpdir() -> None:
 from .utils import (
     cmdstan_path,
     cmdstan_version,
+    disable_logging,
+    enable_logging,
     install_cmdstan,
     set_cmdstan_path,
     set_make_env,
     show_versions,
     write_stan_json,
-    enable_logging,
-    disable_logging,
 )
 
 __all__ = [

cmdstanpy/cmdstan_args.py

@@ -165,7 +165,8 @@ def validate_stanc_opts(self) -> None:
                     del self._stanc_options[deprecated]
                 else:
                     get_logger().warning(
-                        'compiler option "%s" is deprecated and should not be used',
+                        'compiler option "%s" is deprecated and should '
+                        'not be used',
                         deprecated,
                     )
         for key, val in self._stanc_options.items():
@@ -224,7 +225,8 @@ def validate_cpp_opts(self) -> None:
                 val = self._cpp_options[key]
                 if not isinstance(val, int) or val < 0:
                     raise ValueError(
-                        f'{key} must be a non-negative integer value, found {val}.'
+                        f'{key} must be a non-negative integer '
+                        f'value, found {val}.'
                     )
 
     def validate_user_header(self) -> None:
@@ -234,7 +236,8 @@ def validate_user_header(self) -> None:
         """
         if self._user_header != "":
             if not (
-                os.path.exists(self._user_header) and os.path.isfile(self._user_header)
+                os.path.exists(self._user_header)
+                and os.path.isfile(self._user_header)
             ):
                 raise ValueError(
                     f"User header file {self._user_header} cannot be found"
@@ -272,7 +275,9 @@ def add(self, new_opts: "CompilerOptions") -> None:  # noqa: disable=Q000
             else:
                 for key, val in new_opts.stanc_options.items():
                     if key == 'include-paths':
-                        if isinstance(val, Iterable) and not isinstance(val, str):
+                        if isinstance(val, Iterable) and not isinstance(
+                            val, str
+                        ):
                             for path in val:
                                 self.add_include_path(str(path))
                         else:
@@ -337,7 +342,9 @@ def compose(self, filename_in_msg: Optional[str] = None) -> list[str]:
         return opts
 
 
-def src_info(stan_file: str, compiler_options: CompilerOptions) -> dict[str, Any]:
+def src_info(
+    stan_file: str, compiler_options: CompilerOptions
+) -> dict[str, Any]:
     """
     Get source info for Stan program file.
 
@@ -525,7 +532,9 @@ def format_stan_file(
                 else:
                     raise ValueError(
                         "Invalid arguments passed for current CmdStan"
-                        + " version({})\n".format(cmdstan_version() or "Unknown")
+                        + " version({})\n".format(
+                            cmdstan_version() or "Unknown"
+                        )
                         + "--canonicalize requires 2.29 or higher"
                     )
             else:

cmdstanpy/compilation.py

@@ -110,7 +110,9 @@ def latest_version() -> str:
                 print('retry ({}/5)'.format(i + 1))
                 sleep(1)
                 continue
-            raise CmdStanRetrieveError('Cannot connect to CmdStan github repo.') from e
+            raise CmdStanRetrieveError(
+                'Cannot connect to CmdStan github repo.'
+            ) from e
     content = json.loads(response.decode('utf-8'))
     tag = content['tag_name']
     match = re.search(r'v?(.+)', tag)
@@ -286,18 +288,26 @@ def build(verbose: bool = False, progress: bool = True, cores: int = 1) -> None:
         raise CmdStanInstallError(f'Command "make build" failed\n{str(e)}')
     if not os.path.exists(os.path.join('bin', 'stansummary' + EXTENSION)):
         raise CmdStanInstallError(
-            f'bin/stansummary{EXTENSION} not found, please rebuild or report a bug!'
+            f'bin/stansummary{EXTENSION} not found, please rebuild or '
+            'report a bug!'
         )
     if not os.path.exists(os.path.join('bin', 'diagnose' + EXTENSION)):
         raise CmdStanInstallError(
-            f'bin/stansummary{EXTENSION} not found, please rebuild or report a bug!'
+            f'bin/stansummary{EXTENSION} not found, please rebuild or '
+            'report a bug!'
         )
 
     if is_windows():
         # Add tbb to the $PATH on Windows
-        libtbb = os.path.join(os.getcwd(), 'stan', 'lib', 'stan_math', 'lib', 'tbb')
+        libtbb = os.path.join(
+            os.getcwd(), 'stan', 'lib', 'stan_math', 'lib', 'tbb'
+        )
         os.environ['PATH'] = ';'.join(
-            list(OrderedDict.fromkeys([libtbb] + os.environ.get('PATH', '').split(';')))
+            list(
+                OrderedDict.fromkeys(
+                    [libtbb] + os.environ.get('PATH', '').split(';')
+                )
+            )
         )
 
 
@@ -408,9 +418,8 @@ def install_version(
         )
         if overwrite and os.path.exists('.'):
             print(
-                'Overwrite requested, remove existing build of version {}'.format(
-                    cmdstan_version
-                )
+                'Overwrite requested, remove existing build '
+                'of version {}'.format(cmdstan_version)
             )
             clean_all(verbose)
             print('Rebuilding version {}'.format(cmdstan_version))
@@ -477,9 +486,9 @@ def retrieve_version(version: str, progress: bool = True) -> None:
     for i in range(6):  # always retry to allow for transient URLErrors
         try:
             if progress and progbar.allow_show_progress():
-                progress_hook: Optional[Callable[[int, int, int], None]] = (
-                    wrap_url_progress_hook()
-                )
+                progress_hook: Optional[
+                    Callable[[int, int, int], None]
+                ] = wrap_url_progress_hook()
             else:
                 progress_hook = None
             file_tmp, _ = urllib.request.urlretrieve(
@@ -488,13 +497,14 @@ def retrieve_version(version: str, progress: bool = True) -> None:
             break
         except urllib.error.HTTPError as e:
             raise CmdStanRetrieveError(
-                'HTTPError: {}\nVersion {} not available from github.com.'.format(
-                    e.code, version
-                )
+                'HTTPError: {}\nVersion {} not available from '
+                'github.com.'.format(e.code, version)
             ) from e
         except urllib.error.URLError as e:
             print(
-                'Failed to download CmdStan version {} from github.com'.format(version)
+                'Failed to download CmdStan version {} from github.com'.format(
+                    version
+                )
             )
             print(e)
             if i < 5:
@@ -640,7 +650,8 @@ def parse_cmdline_args() -> dict[str, Any]:
         '--interactive',
         '-i',
         action='store_true',
-        help="Ignore other arguments and run the installation in " + "interactive mode",
+        help="Ignore other arguments and run the installation in "
+        + "interactive mode",
     )
     parser.add_argument(
         '--version',

cmdstanpy/install_cmdstan.py

@@ -244,9 +244,7 @@ def get_url(version: str) -> str:
         if version == '4.0':
             # pylint: disable=line-too-long
             if IS_64BITS:
-                url = (
-                    'https://cran.r-project.org/bin/windows/Rtools/rtools40-x86_64.exe'  # noqa: disable=E501
-                )
+                url = 'https://cran.r-project.org/bin/windows/Rtools/rtools40-x86_64.exe'  # noqa: disable=E501
             else:
                 url = 'https://cran.r-project.org/bin/windows/Rtools/rtools40-i686.exe'  # noqa: disable=E501
         elif version == '3.5':
@@ -311,7 +309,9 @@ def run_rtools_install(args: dict[str, Any]) -> None:
         else:
             if os.path.exists(toolchain_folder):
                 shutil.rmtree(toolchain_folder, ignore_errors=False)
-            retrieve_toolchain(toolchain_folder + EXTENSION, url, progress=progress)
+            retrieve_toolchain(
+                toolchain_folder + EXTENSION, url, progress=progress
+            )
             install_version(
                 toolchain_folder,
                 toolchain_folder + EXTENSION,
@@ -325,7 +325,9 @@ def run_rtools_install(args: dict[str, Any]) -> None:
             and (version in ('4.0', '4', '40'))
         ):
             if os.path.exists(
-                os.path.join(toolchain_folder, 'mingw64', 'bin', 'mingw32-make.exe')
+                os.path.join(
+                    toolchain_folder, 'mingw64', 'bin', 'mingw32-make.exe'
+                )
             ):
                 print('mingw32-make.exe already installed')
             else: