Remove --container argument from CLIs

cubids/__init__.py

@@ -1,15 +1,6 @@
 """Top-level package for CuBIDS."""
 
-from cubids import (
-    cli,
-    config,
-    constants,
-    cubids,
-    metadata_merge,
-    utils,
-    validator,
-    workflows,
-)
+from cubids import cli, config, constants, cubids, metadata_merge, validator, workflows
 from cubids.__about__ import __copyright__, __credits__, __packagename__, __version__
 
 __all__ = [
@@ -22,7 +13,6 @@
     "constants",
     "cubids",
     "metadata_merge",
-    "utils",
     "validator",
     "workflows",
 ]

cubids/cli.py

@@ -105,12 +105,6 @@ def _parse_validate():
         help="Run the BIDS validator sequentially on each subject.",
         required=False,
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-        default=None,
-    )
     parser.add_argument(
         "--ignore-nifti-headers",
         action="store_true",
@@ -251,11 +245,6 @@ def _parse_group():
             "then output files will go to the specified location."
         ),
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
     parser.add_argument(
         "--acq-group-level",
         default="subject",
@@ -347,11 +336,6 @@ def _parse_apply():
         default=False,
         help="ensure that there are no untracked changes before finding groups",
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
     parser.add_argument(
         "--acq-group-level",
         default="subject",
@@ -405,11 +389,6 @@ def _parse_datalad_save():
         action="store",
         help="message for this commit",
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
 
     return parser
 
@@ -442,11 +421,6 @@ def _parse_undo():
             "sub-X directories and dataset_description.json"
         ),
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
 
     return parser
 
@@ -532,11 +506,6 @@ def _parse_copy_exemplars():
     #                     help='only include an exemplar subject from these '
     #                     'listed Acquisition Groups in the exemplar dataset ',
     #                     required=False)
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
     parser.add_argument(
         "--force-unlock",
         action="store_true",
@@ -590,11 +559,6 @@ def _parse_add_nifti_info():
         default=False,
         help="unlock dataset before adding nifti info ",
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
     return parser
 
 
@@ -645,11 +609,6 @@ def _parse_purge():
         default=False,
         help="ensure that there are no untracked changes before finding groups",
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
     return parser
 
 
@@ -687,11 +646,6 @@ def _parse_remove_metadata_fields():
         default=[],
         help="space-separated list of metadata fields to remove.",
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
 
     return parser
 
@@ -726,11 +680,6 @@ def _parse_print_metadata_fields():
             "sub-X directories and dataset_description.json"
         ),
     )
-    parser.add_argument(
-        "--container",
-        action="store",
-        help="Docker image tag or Singularity image file.",
-    )
 
     return parser
 

cubids/config.py

@@ -1,9 +1,11 @@
 """Functions for configuring CuBIDS."""
 
-from pathlib import Path
 import importlib.resources
+from pathlib import Path
+
 import yaml
 
+
 def load_config(config_file):
     """Load a YAML file containing a configuration for param groups.
 

cubids/tests/test_apply.py

@@ -300,7 +300,6 @@ def test_cubids_apply_intendedfor(
             edited_summary_tsv=summary_tsv,
             files_tsv=files_tsv,
             new_tsv_prefix=None,
-            container=None,
         )
 
         with open(fmap_json) as f:
@@ -317,5 +316,4 @@ def test_cubids_apply_intendedfor(
                 edited_summary_tsv=summary_tsv,
                 files_tsv=files_tsv,
                 new_tsv_prefix=None,
-                container=None,
             )

cubids/tests/test_bond.py

@@ -24,13 +24,13 @@
     get_data,
 )
 from cubids.validator import (
+    bids_validator_version,
     build_validator_call,
+    extract_summary_info,
+    get_bids_validator_version,
     parse_validator_output,
     run_validator,
-    get_bids_validator_version,
-    extract_summary_info,
     update_dataset_description,
-    bids_validator_version,
 )
 
 COMPLETE_KEY_GROUPS = [

cubids/tests/utils.py

@@ -1,6 +1,7 @@
 """Utility functions for CuBIDS' tests."""
 
 import hashlib
+import importlib.resources
 import json
 import os
 import shutil
@@ -10,7 +11,6 @@
 import nibabel as nb
 import numpy as np
 import pandas as pd
-import importlib.resources
 
 TEST_DATA = importlib.resources.files("cubids") / "tests/data"