junifer 0.0.5.dev242__py3-none-any.whl → 0.0.6__py3-none-any.whl

junifer/cli/__init__.py

@@ -0,0 +1,24 @@
+"""Junifer CLI components."""
+
+# Authors: Synchon Mandal <s.mandal@fz-juelich.de>
+# License: AGPL
+
+import sys
+
+import lazy_loader as lazy
+
+
+if sys.version_info < (3, 11):  # pragma: no cover
+    from importlib_metadata import entry_points
+else:
+    from importlib.metadata import entry_points
+
+
+__getattr__, __dir__, __all__ = lazy.attach_stub(__name__, __file__)
+
+
+# Register extensions
+from .cli import cli
+
+for ep in entry_points(group="junifer.ext"):
+    cli.add_command(ep.load(), name=ep.name)

junifer/cli/__init__.pyi

@@ -0,0 +1,3 @@
+__all__ = ["cli"]
+
+from .cli import cli

junifer/{api → cli}/cli.py

@@ -1,4 +1,4 @@
-"""Provide functions for cli."""
+"""Provide functions for CLI."""
 
 # Authors: Federico Raimondo <f.raimondo@fz-juelich.de>
 #          Synchon Mandal <s.mandal@fz-juelich.de>
@@ -8,120 +8,67 @@ import pathlib
 import subprocess
 import sys
 from pathlib import Path
-from typing import Dict, List, Optional, Tuple, Union
+from typing import Optional, Union
 
 import click
-import pandas as pd
 
-from ..utils.logging import (
+from ..api import functions as cli_func
+from ..utils import (
     configure_logging,
-    logger,
     raise_error,
-    warn_with_log,
+    yaml,
 )
-from .functions import collect as api_collect
-from .functions import list_elements as api_list_elements
-from .functions import queue as api_queue
-from .functions import reset as api_reset
-from .functions import run as api_run
-from .parser import parse_yaml
+from .parser import parse_elements, parse_yaml
 from .utils import (
     _get_dependency_information,
     _get_environment_information,
     _get_junifer_version,
     _get_python_information,
     _get_system_information,
-    yaml,
 )
 
 
-def _parse_elements(element: Tuple[str], config: Dict) -> Union[List, None]:
-    """Parse elements from cli.
+__all__ = [
+    "afni_docker",
+    "ants_docker",
+    "cli",
+    "collect",
+    "freesurfer_docker",
+    "fsl_docker",
+    "list_elements",
+    "queue",
+    "reset",
+    "run",
+    "selftest",
+    "setup",
+    "wtf",
+]
+
+
+def _validate_optional_verbose(
+    ctx: click.Context, param: str, value: Optional[str]
+):
+    """Validate optional verbose option.
 
     Parameters
     ----------
-    element : tuple of str
-        The element(s) to operate on.
-    config : dict
-        The configuration to operate using.
+    ctx : click.Context
+        The context of the command.
+    param : str
+        The parameter to validate.
+    value : str
+        The value to validate.
 
     Returns
     -------
-    list or None
-        The element(s) as list or None.
-
-    Raises
-    ------
-    ValueError
-        If no element is found either in the command-line options or
-        the configuration file.
-
-    Warns
-    -----
-    RuntimeWarning
-        If elements are specified both via the command-line options and
-        the configuration file.
+    str or int or None
+        The validated value.
 
     """
-    logger.debug(f"Parsing elements: {element}")
-    # Early return None to continue with all elements
-    if len(element) == 0:
-        return None
-    # Check if the element is a file for single element;
-    # if yes, then parse elements from it
-    if len(element) == 1 and Path(element[0]).resolve().is_file():
-        elements = _parse_elements_file(Path(element[0]).resolve())
+    if value is None:
+        return value
     else:
-        # Process multi-keyed elements
-        elements = [tuple(x.split(",")) if "," in x else x for x in element]
-    logger.debug(f"Parsed elements: {elements}")
-    if elements is not None and "elements" in config:
-        warn_with_log(
-            "One or more elements have been specified in both the command "
-            "line and in the config file. The command line has precedence "
-            "over the configuration file. That is, the elements specified "
-            "in the command line will be used. The elements specified in "
-            "the configuration file will be ignored. To remove this warning, "
-            "please remove the `elements` item from the configuration file."
-        )
-    elif elements is None:
-        # Check in config
-        elements = config.get("elements", None)
-        if elements is None:
-            raise_error(
-                "The `elements` key is set in the configuration, but its value"
-                " is `None`. It is likely that there is an empty `elements` "
-                "section in the yaml configuration file."
-            )
-    return elements
-
-
-def _parse_elements_file(filepath: Path) -> List[Tuple[str, ...]]:
-    """Parse elements from file.
-
-    Parameters
-    ----------
-    filepath : pathlib.Path
-        The path to the element file.
-
-    Returns
-    -------
-    list of tuple of str
-        The element(s) as list.
-
-    """
-    # Read CSV into dataframe
-    csv_df = pd.read_csv(
-        filepath,
-        header=None,  # no header  # type: ignore
-        index_col=False,  # no index column
-        dtype=str,
-        skipinitialspace=True,  # no leading space after delimiter
-    )
-    # Remove trailing whitespace in cell entries
-    csv_df_trimmed = csv_df.apply(lambda x: x.str.strip())
-    # Convert to list of tuple of str
-    return list(map(tuple, csv_df_trimmed.to_numpy()))
+        return _validate_verbose(ctx, param, value)
 
 
 def _validate_verbose(
@@ -164,9 +111,11 @@ def _validate_verbose(
     )
 
 
-@click.group()
+@click.group
+@click.version_option()
+@click.help_option()
 def cli() -> None:  # pragma: no cover
-    """CLI for JUelich NeuroImaging FEature extractoR."""
+    """JUelich NeuroImaging FEature extractoR."""
 
 
 @cli.command()
@@ -184,10 +133,19 @@ def cli() -> None:  # pragma: no cover
     callback=_validate_verbose,
     default="info",
 )
+@click.option(
+    "--verbose-datalad",
+    type=click.UNPROCESSED,
+    callback=_validate_optional_verbose,
+    default=None,
+)
 def run(
-    filepath: click.Path, element: Tuple[str], verbose: Union[str, int]
+    filepath: click.Path,
+    element: tuple[str],
+    verbose: Union[str, int],
+    verbose_datalad: Optional[Union[str, int]],
 ) -> None:
-    """Run command for CLI.
+    """Run feature extraction.
 
     \f
 
@@ -196,15 +154,18 @@ def run(
     filepath : click.Path
         The filepath to the configuration file.
     element : tuple of str
-        The element to operate on.
+        The element(s) to operate on.
     verbose : click.Choice
         The verbosity level: warning, info or debug (default "info").
+    verbose_datalad : click.Choice or None
+        The verbosity level for datalad: warning, info or debug (default None).
 
     """
-    configure_logging(level=verbose)
+    # Setup logging
+    configure_logging(level=verbose, level_datalad=verbose_datalad)
     # TODO(synchon): add validation
     # Parse YAML
-    config = parse_yaml(filepath)  # type: ignore
+    config = parse_yaml(filepath)
     # Retrieve working directory
     workdir = config["workdir"]
     # Fetch datagrabber
@@ -219,9 +180,9 @@ def run(
     if preprocessors is not None and not isinstance(preprocessors, list):
         preprocessors = [preprocessors]
     # Parse elements
-    elements = _parse_elements(element, config)
+    elements = parse_elements(element, config)
     # Perform operation
-    api_run(
+    cli_func.run(
         workdir=workdir,
         datagrabber=datagrabber,
         markers=markers,
@@ -245,8 +206,18 @@ def run(
     callback=_validate_verbose,
     default="info",
 )
-def collect(filepath: click.Path, verbose: Union[str, int]) -> None:
-    """Collect command for CLI.
+@click.option(
+    "--verbose-datalad",
+    type=click.UNPROCESSED,
+    callback=_validate_optional_verbose,
+    default=None,
+)
+def collect(
+    filepath: click.Path,
+    verbose: Union[str, int],
+    verbose_datalad: Union[str, int, None],
+) -> None:
+    """Collect extracted features.
 
     \f
 
@@ -256,14 +227,19 @@ def collect(filepath: click.Path, verbose: Union[str, int]) -> None:
         The filepath to the configuration file.
     verbose : click.Choice
         The verbosity level: warning, info or debug (default "info").
+    verbose_datalad : click.Choice or None
+        The verbosity level for datalad: warning, info or debug (default None).
 
     """
-    configure_logging(level=verbose)
+    # Setup logging
+    configure_logging(level=verbose, level_datalad=verbose_datalad)
     # TODO: add validation
-    config = parse_yaml(filepath)  # type: ignore
+    # Parse YAML
+    config = parse_yaml(filepath)
+    # Fetch storage
     storage = config["storage"]
     # Perform operation
-    api_collect(storage=storage)
+    cli_func.collect(storage=storage)
 
 
 @cli.command()
@@ -283,14 +259,21 @@ def collect(filepath: click.Path, verbose: Union[str, int]) -> None:
     callback=_validate_verbose,
     default="info",
 )
+@click.option(
+    "--verbose-datalad",
+    type=click.UNPROCESSED,
+    callback=_validate_optional_verbose,
+    default=None,
+)
 def queue(
     filepath: click.Path,
-    element: str,
+    element: tuple[str],
     overwrite: bool,
     submit: bool,
     verbose: Union[str, int],
+    verbose_datalad: Union[str, int, None],
 ) -> None:
-    """Queue command for CLI.
+    """Queue feature extraction.
 
     \f
 
@@ -298,25 +281,38 @@ def queue(
     ----------
     filepath : click.Path
         The filepath to the configuration file.
-    element : str
-        The element to operate using.
+    element : tuple of str
+        The element(s) to operate on.
     overwrite : bool
         Whether to overwrite existing directory.
     submit : bool
         Whether to submit the job.
     verbose : click.Choice
         The verbosity level: warning, info or debug (default "info").
+    verbose_datalad : click.Choice or None
+        The verbosity level for datalad: warning, info or debug (default None).
+
+    Raises
+    ------
+    ValueError
+        If no ``queue`` section is found in the YAML.
 
     """
-    configure_logging(level=verbose)
+    # Setup logging
+    configure_logging(level=verbose, level_datalad=verbose_datalad)
     # TODO: add validation
+    # Parse YAML
     config = parse_yaml(filepath)  # type: ignore
-    elements = _parse_elements(element, config)
+    # Check queue section
     if "queue" not in config:
         raise_error(f"No queue configuration found in {filepath}.")
+    # Parse elements
+    elements = parse_elements(element, config)
+    # Separate out queue section
     queue_config = config.pop("queue")
     kind = queue_config.pop("kind")
-    api_queue(
+    # Perform operation
+    cli_func.queue(
         config=config,
         kind=kind,
         overwrite=overwrite,
@@ -329,7 +325,7 @@ def queue(
 @cli.command()
 @click.option("--long", "long_", is_flag=True)
 def wtf(long_: bool) -> None:
-    """Wtf command for CLI.
+    """WTF?!.
 
     \f
 
@@ -352,7 +348,7 @@ def wtf(long_: bool) -> None:
 @cli.command()
 @click.argument("subpkg", type=str)
 def selftest(subpkg: str) -> None:
-    """Selftest command for CLI.
+    """Selftest.
 
     \f
 
@@ -429,11 +425,18 @@ def selftest(subpkg: str) -> None:
     callback=_validate_verbose,
     default="info",
 )
+@click.option(
+    "--verbose-datalad",
+    type=click.UNPROCESSED,
+    callback=_validate_optional_verbose,
+    default=None,
+)
 def reset(
     filepath: click.Path,
     verbose: Union[str, int],
+    verbose_datalad: Union[str, int, None],
 ) -> None:
-    """Reset command for CLI.
+    """Reset generated assets.
 
     \f
 
@@ -443,13 +446,16 @@ def reset(
         The filepath to the configuration file.
     verbose : click.Choice
         The verbosity level: warning, info or debug (default "info").
+    verbose_datalad : click.Choice or None
+        The verbosity level for datalad: warning, info or debug (default None).
 
     """
-    configure_logging(level=verbose)
+    # Setup logging
+    configure_logging(level=verbose, level_datalad=verbose_datalad)
     # Parse YAML
     config = parse_yaml(filepath)
     # Perform operation
-    api_reset(config)
+    cli_func.reset(config)
 
 
 @cli.command()
@@ -472,13 +478,20 @@ def reset(
     callback=_validate_verbose,
     default="info",
 )
+@click.option(
+    "--verbose-datalad",
+    type=click.UNPROCESSED,
+    callback=_validate_optional_verbose,
+    default=None,
+)
 def list_elements(
     filepath: click.Path,
-    element: Tuple[str],
+    element: tuple[str],
     output_file: Optional[click.Path],
     verbose: Union[str, int],
+    verbose_datalad: Union[str, int, None],
 ) -> None:
-    """Element listing command for CLI.
+    """List elements of a dataset.
 
     \f
 
@@ -487,23 +500,26 @@ def list_elements(
     filepath : click.Path
         The filepath to the configuration file.
     element : tuple of str
-        The element to operate on.
+        The element(s) to operate on.
     output_file : click.Path or None
         The path to write the output to. If not None, writing to
         stdout is not performed.
     verbose : click.Choice
         The verbosity level: warning, info or debug (default "info").
+    verbose_datalad : click.Choice or None
+        The verbosity level for datalad: warning, info or debug (default None
 
     """
-    configure_logging(level=verbose)
+    # Setup logging
+    configure_logging(level=verbose, level_datalad=verbose_datalad)
     # Parse YAML
-    config = parse_yaml(filepath)  # type: ignore
+    config = parse_yaml(filepath)
     # Fetch datagrabber
     datagrabber = config["datagrabber"]
     # Parse elements
-    elements = _parse_elements(element, config)
+    elements = parse_elements(element, config)
     # Perform operation
-    listed_elements = api_list_elements(
+    listed_elements = cli_func.list_elements(
         datagrabber=datagrabber,
         elements=elements,
     )
@@ -517,7 +533,7 @@ def list_elements(
 
 @cli.group()
 def setup() -> None:  # pragma: no cover
-    """Configure commands for Junifer."""
+    """Configure external tools."""
     pass