umu-commander 1.5.2__tar.gz → 1.5.4__tar.gz

{umu_commander-1.5.2 → umu_commander-1.5.4}/.idea/workspace.xml

@@ -4,22 +4,9 @@
     <option name="autoReloadType" value="SELECTIVE" />
   </component>
   <component name="ChangeListManager">
-    <list default="true" id="043620f1-711f-4c27-a2e2-15fa501e8ce0" name="Changes" comment="Preparations for building">
-      <change afterPath="$PROJECT_DIR$/src/umu_commander/config.py" afterDir="false" />
-      <change afterPath="$PROJECT_DIR$/tests/test_config.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/.idea/misc.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/misc.xml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/.idea/umu-commander.iml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/umu-commander.iml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/pyproject.toml" beforeDir="false" afterPath="$PROJECT_DIR$/pyproject.toml" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/umu_commander/configuration.py" beforeDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/umu_commander/db.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/db.py" afterDir="false" />
+    <list default="true" id="043620f1-711f-4c27-a2e2-15fa501e8ce0" name="Changes" comment="1.5.3">
       <change beforePath="$PROJECT_DIR$/src/umu_commander/main.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/main.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/umu_commander/proton.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/proton.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/umu_commander/tracking.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/tracking.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/umu_commander/umu_config.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/umu_config.py" afterDir="false" />
       <change beforePath="$PROJECT_DIR$/src/umu_commander/util.py" beforeDir="false" afterPath="$PROJECT_DIR$/src/umu_commander/util.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/tests/test_db.py" beforeDir="false" afterPath="$PROJECT_DIR$/tests/test_db.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/tests/test_proton.py" beforeDir="false" afterPath="$PROJECT_DIR$/tests/test_proton.py" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/tests/test_tracking.py" beforeDir="false" afterPath="$PROJECT_DIR$/tests/test_tracking.py" afterDir="false" />
     </list>
     <option name="SHOW_DIALOG" value="false" />
     <option name="HIGHLIGHT_CONFLICTS" value="true" />
@@ -75,6 +62,7 @@
     &quot;Python tests.Python tests for test_proton.Tracking.test_collect_proton_versions.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests for test_tracking.Tracking.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests for test_tracking.Tracking.test_track.executor&quot;: &quot;Run&quot;,
+    &quot;Python tests.Python tests for test_tracking.Tracking.test_track_untrack.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests for test_tracking.TrackingTest.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests for test_tracking.TrackingTest.test_track.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests for tests.test_db.DB.executor&quot;: &quot;Run&quot;,
@@ -87,7 +75,7 @@
     &quot;Python tests.Python tests for tests.test_tracking.TrackingTest.test_track.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Python tests in tests.executor&quot;: &quot;Run&quot;,
     &quot;Python tests.Run tests.executor&quot;: &quot;Run&quot;,
-    &quot;Python.create.executor&quot;: &quot;Debug&quot;,
+    &quot;Python.create.executor&quot;: &quot;Run&quot;,
     &quot;Python.delete.executor&quot;: &quot;Run&quot;,
     &quot;Python.manage.executor&quot;: &quot;Run&quot;,
     &quot;Python.run.executor&quot;: &quot;Run&quot;,
@@ -114,7 +102,7 @@
       <recent name="$PROJECT_DIR$/src" />
     </key>
   </component>
-  <component name="RunManager" selected="Python.run">
+  <component name="RunManager" selected="Python.users">
     <configuration name="create" type="PythonConfigurationType" factoryName="Python">
       <module name="umu-commander" />
       <option name="ENV_FILES" value="" />
@@ -248,7 +236,7 @@
       <method v="2" />
     </configuration>
     <configuration name="Build" type="ShConfigurationType">
-      <option name="SCRIPT_TEXT" value="rm dist/* &amp;&amp; python3 -m build" />
+      <option name="SCRIPT_TEXT" value="rm dist/*; python3 -m build" />
       <option name="INDEPENDENT_SCRIPT_PATH" value="true" />
       <option name="SCRIPT_PATH" value="" />
       <option name="SCRIPT_OPTIONS" value="" />
@@ -309,6 +297,36 @@
       <option name="_new_targetType" value="&quot;PYTHON&quot;" />
       <method v="2" />
     </configuration>
+    <configuration name="Python tests for test_proton.Tracking" type="tests" factoryName="Autodetect" temporary="true" nameIsGenerated="true">
+      <module name="umu-commander" />
+      <option name="ENV_FILES" value="" />
+      <option name="INTERPRETER_OPTIONS" value="" />
+      <option name="PARENT_ENVS" value="true" />
+      <option name="SDK_HOME" value="" />
+      <option name="WORKING_DIRECTORY" value="$PROJECT_DIR$/tests" />
+      <option name="IS_MODULE_SDK" value="true" />
+      <option name="ADD_CONTENT_ROOTS" value="true" />
+      <option name="ADD_SOURCE_ROOTS" value="true" />
+      <option name="_new_additionalArguments" value="&quot;&quot;" />
+      <option name="_new_target" value="&quot;test_proton.Tracking&quot;" />
+      <option name="_new_targetType" value="&quot;PYTHON&quot;" />
+      <method v="2" />
+    </configuration>
+    <configuration name="Python tests for test_tracking.Tracking.test_track_untrack" type="tests" factoryName="Autodetect" temporary="true" nameIsGenerated="true">
+      <module name="umu-commander" />
+      <option name="ENV_FILES" value="" />
+      <option name="INTERPRETER_OPTIONS" value="" />
+      <option name="PARENT_ENVS" value="true" />
+      <option name="SDK_HOME" value="" />
+      <option name="WORKING_DIRECTORY" value="$PROJECT_DIR$/tests" />
+      <option name="IS_MODULE_SDK" value="true" />
+      <option name="ADD_CONTENT_ROOTS" value="true" />
+      <option name="ADD_SOURCE_ROOTS" value="true" />
+      <option name="_new_additionalArguments" value="&quot;&quot;" />
+      <option name="_new_target" value="&quot;test_tracking.Tracking.test_track_untrack&quot;" />
+      <option name="_new_targetType" value="&quot;PYTHON&quot;" />
+      <method v="2" />
+    </configuration>
     <configuration name="Run tests" type="tests" factoryName="Autodetect">
       <module name="umu-commander" />
       <option name="ENV_FILES" value="" />
@@ -332,6 +350,8 @@
       <item itemvalue="Python.delete" />
       <item itemvalue="Python.create" />
       <item itemvalue="Python tests.Run tests" />
+      <item itemvalue="Python tests.Python tests for test_proton.Tracking" />
+      <item itemvalue="Python tests.Python tests for test_tracking.Tracking.test_track_untrack" />
       <item itemvalue="Python tests.Python tests for test_config.Config" />
       <item itemvalue="Python tests.Python tests for test_config.Config.test_missing_config" />
       <item itemvalue="Shell Script.Upload" />
@@ -339,6 +359,8 @@
     </list>
     <recent_temporary>
       <list>
+        <item itemvalue="Python tests.Python tests for test_tracking.Tracking.test_track_untrack" />
+        <item itemvalue="Python tests.Python tests for test_proton.Tracking" />
         <item itemvalue="Python tests.Python tests for test_config.Config" />
         <item itemvalue="Python tests.Python tests for test_config.Config.test_missing_config" />
       </list>
@@ -360,86 +382,6 @@
       <option name="presentableId" value="Default" />
       <updated>1751043400140</updated>
     </task>
-    <task id="LOCAL-00001" summary="Initial implementation">
-      <option name="closed" value="true" />
-      <created>1751130219943</created>
-      <option name="number" value="00001" />
-      <option name="presentableId" value="LOCAL-00001" />
-      <option name="project" value="LOCAL" />
-      <updated>1751130219943</updated>
-    </task>
-    <task id="LOCAL-00002" summary="Move configuration to separate file">
-      <option name="closed" value="true" />
-      <created>1751917068399</created>
-      <option name="number" value="00002" />
-      <option name="presentableId" value="LOCAL-00002" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917068399</updated>
-    </task>
-    <task id="LOCAL-00003" summary="Move DB to separate file">
-      <option name="closed" value="true" />
-      <created>1751917093739</created>
-      <option name="number" value="00003" />
-      <option name="presentableId" value="LOCAL-00003" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917093739</updated>
-    </task>
-    <task id="LOCAL-00004" summary="Split functionality into multiple files&#10;Rework verbs&#10;Add functionality&#10;Reword existing functionality">
-      <option name="closed" value="true" />
-      <created>1751917142976</created>
-      <option name="number" value="00004" />
-      <option name="presentableId" value="LOCAL-00004" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917142976</updated>
-    </task>
-    <task id="LOCAL-00005" summary="Define classes for better readability">
-      <option name="closed" value="true" />
-      <created>1751917157187</created>
-      <option name="number" value="00005" />
-      <option name="presentableId" value="LOCAL-00005" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917157187</updated>
-    </task>
-    <task id="LOCAL-00006" summary="pycharm stuff">
-      <option name="closed" value="true" />
-      <created>1751917165409</created>
-      <option name="number" value="00006" />
-      <option name="presentableId" value="LOCAL-00006" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917165409</updated>
-    </task>
-    <task id="LOCAL-00007" summary="Add deletion warning">
-      <option name="closed" value="true" />
-      <created>1751917368353</created>
-      <option name="number" value="00007" />
-      <option name="presentableId" value="LOCAL-00007" />
-      <option name="project" value="LOCAL" />
-      <updated>1751917368353</updated>
-    </task>
-    <task id="LOCAL-00008" summary="Update readme">
-      <option name="closed" value="true" />
-      <created>1751918258510</created>
-      <option name="number" value="00008" />
-      <option name="presentableId" value="LOCAL-00008" />
-      <option name="project" value="LOCAL" />
-      <updated>1751918258510</updated>
-    </task>
-    <task id="LOCAL-00009" summary="Update readme and decapitalise umu">
-      <option name="closed" value="true" />
-      <created>1751918299613</created>
-      <option name="number" value="00009" />
-      <option name="presentableId" value="LOCAL-00009" />
-      <option name="project" value="LOCAL" />
-      <updated>1751918299613</updated>
-    </task>
-    <task id="LOCAL-00010" summary="Add example config">
-      <option name="closed" value="true" />
-      <created>1751919191540</created>
-      <option name="number" value="00010" />
-      <option name="presentableId" value="LOCAL-00010" />
-      <option name="project" value="LOCAL" />
-      <updated>1751919191540</updated>
-    </task>
     <task id="LOCAL-00011" summary="Tweak &quot;latest umu&quot; functionality">
       <option name="closed" value="true" />
       <created>1751919211128</created>
@@ -712,7 +654,127 @@
       <option name="project" value="LOCAL" />
       <updated>1753347983462</updated>
     </task>
-    <option name="localTasksCounter" value="45" />
+    <task id="LOCAL-00045" summary="Break off configuration">
+      <option name="closed" value="true" />
+      <created>1754002639114</created>
+      <option name="number" value="00045" />
+      <option name="presentableId" value="LOCAL-00045" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002639114</updated>
+    </task>
+    <task id="LOCAL-00046" summary="Rework DB">
+      <option name="closed" value="true" />
+      <created>1754002669820</created>
+      <option name="number" value="00046" />
+      <option name="presentableId" value="LOCAL-00046" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002669820</updated>
+    </task>
+    <task id="LOCAL-00047" summary="Create convenience classes">
+      <option name="closed" value="true" />
+      <created>1754002693811</created>
+      <option name="number" value="00047" />
+      <option name="presentableId" value="LOCAL-00047" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002693811</updated>
+    </task>
+    <task id="LOCAL-00048" summary="Cleanup return codes and config/db handling">
+      <option name="closed" value="true" />
+      <created>1754002709864</created>
+      <option name="number" value="00048" />
+      <option name="presentableId" value="LOCAL-00048" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002709864</updated>
+    </task>
+    <task id="LOCAL-00049" summary="Improvements to proton handling methods">
+      <option name="closed" value="true" />
+      <created>1754002728560</created>
+      <option name="number" value="00049" />
+      <option name="presentableId" value="LOCAL-00049" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002728560</updated>
+    </task>
+    <task id="LOCAL-00050" summary="Use tomli-w&#10;Improvements to umu config creation">
+      <option name="closed" value="true" />
+      <created>1754002792462</created>
+      <option name="number" value="00050" />
+      <option name="presentableId" value="LOCAL-00050" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002792462</updated>
+    </task>
+    <task id="LOCAL-00051" summary="Cleanup">
+      <option name="closed" value="true" />
+      <created>1754002812560</created>
+      <option name="number" value="00051" />
+      <option name="presentableId" value="LOCAL-00051" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002812560</updated>
+    </task>
+    <task id="LOCAL-00052" summary="Tests">
+      <option name="closed" value="true" />
+      <created>1754002840695</created>
+      <option name="number" value="00052" />
+      <option name="presentableId" value="LOCAL-00052" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002840695</updated>
+    </task>
+    <task id="LOCAL-00053" summary="Formatting">
+      <option name="closed" value="true" />
+      <created>1754002845468</created>
+      <option name="number" value="00053" />
+      <option name="presentableId" value="LOCAL-00053" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002845468</updated>
+    </task>
+    <task id="LOCAL-00054" summary="1.5.1">
+      <option name="closed" value="true" />
+      <created>1754002864635</created>
+      <option name="number" value="00054" />
+      <option name="presentableId" value="LOCAL-00054" />
+      <option name="project" value="LOCAL" />
+      <updated>1754002864635</updated>
+    </task>
+    <task id="LOCAL-00055" summary="Improve config handling">
+      <option name="closed" value="true" />
+      <created>1754003601420</created>
+      <option name="number" value="00055" />
+      <option name="presentableId" value="LOCAL-00055" />
+      <option name="project" value="LOCAL" />
+      <updated>1754003601421</updated>
+    </task>
+    <task id="LOCAL-00056" summary="Update documentation">
+      <option name="closed" value="true" />
+      <created>1754003609647</created>
+      <option name="number" value="00056" />
+      <option name="presentableId" value="LOCAL-00056" />
+      <option name="project" value="LOCAL" />
+      <updated>1754003609647</updated>
+    </task>
+    <task id="LOCAL-00057" summary="1.5.2">
+      <option name="closed" value="true" />
+      <created>1754003619303</created>
+      <option name="number" value="00057" />
+      <option name="presentableId" value="LOCAL-00057" />
+      <option name="project" value="LOCAL" />
+      <updated>1754003619303</updated>
+    </task>
+    <task id="LOCAL-00058" summary="made typo.">
+      <option name="closed" value="true" />
+      <created>1754004029502</created>
+      <option name="number" value="00058" />
+      <option name="presentableId" value="LOCAL-00058" />
+      <option name="project" value="LOCAL" />
+      <updated>1754004029502</updated>
+    </task>
+    <task id="LOCAL-00059" summary="1.5.3">
+      <option name="closed" value="true" />
+      <created>1754004038591</created>
+      <option name="number" value="00059" />
+      <option name="presentableId" value="LOCAL-00059" />
+      <option name="project" value="LOCAL" />
+      <updated>1754004038591</updated>
+    </task>
+    <option name="localTasksCounter" value="60" />
     <servers />
   </component>
   <component name="Vcs.Log.Tabs.Properties">
@@ -739,21 +801,6 @@
     </option>
   </component>
   <component name="VcsManagerConfiguration">
-    <MESSAGE value="Update internal readme" />
-    <MESSAGE value="Merge remote-tracking branch 'origin/master'" />
-    <MESSAGE value="Formatting" />
-    <MESSAGE value="Specify python version in readme" />
-    <MESSAGE value="Handle vanilla configs too" />
-    <MESSAGE value="Test formatting precommit hook" />
-    <MESSAGE value="Add license" />
-    <MESSAGE value="Update readme to mention umu-run dependency" />
-    <MESSAGE value="Tracking was broken" />
-    <MESSAGE value="Rename Classes file to classes" />
-    <MESSAGE value="Make tracking DB more readable" />
-    <MESSAGE value="Handle missing DB_DIR" />
-    <MESSAGE value="Support multiple proton directories" />
-    <MESSAGE value="Add UMU_PROTON_DIR config key" />
-    <MESSAGE value="Rename DB methods&#10;Rename variables for consistency&#10;Use dataclasses to group data better&#10;Add full support for multiple proton directories" />
     <MESSAGE value="shorter" />
     <MESSAGE value="Add some comments, rename a DB method" />
     <MESSAGE value="Move src code to src directory" />
@@ -764,13 +811,21 @@
     <MESSAGE value="Fix documentation" />
     <MESSAGE value="Add messages for some tests" />
     <MESSAGE value="Preparations for building" />
-    <option name="LAST_COMMIT_MESSAGE" value="Preparations for building" />
-  </component>
-  <component name="XDebuggerManager">
-    <watches-manager>
-      <configuration name="PythonConfigurationType">
-        <watch expression="cls._conf" language="Python" />
-      </configuration>
-    </watches-manager>
+    <MESSAGE value="Break off configuration" />
+    <MESSAGE value="Rework DB" />
+    <MESSAGE value="Create convenience classes" />
+    <MESSAGE value="Cleanup return codes and config/db handling" />
+    <MESSAGE value="Improvements to proton handling methods" />
+    <MESSAGE value="Use tomli-w&#10;Improvements to umu config creation" />
+    <MESSAGE value="Cleanup" />
+    <MESSAGE value="Tests" />
+    <MESSAGE value="Formatting" />
+    <MESSAGE value="1.5.1" />
+    <MESSAGE value="Improve config handling" />
+    <MESSAGE value="Update documentation" />
+    <MESSAGE value="1.5.2" />
+    <MESSAGE value="made typo." />
+    <MESSAGE value="1.5.3" />
+    <option name="LAST_COMMIT_MESSAGE" value="1.5.3" />
   </component>
 </project>

{umu_commander-1.5.2 → umu_commander-1.5.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: umu-commander
-Version: 1.5.2
+Version: 1.5.4
 Summary: umu-commander is an interactive CLI tool to help you manage umu.
 Project-URL: Homepage, https://github.com/Mpaxlamitsounas/umu-commander
 Project-URL: Issues, https://github.com/Mpaxlamitsounas/umu-commander/issues
@@ -11,6 +11,7 @@ Keywords: umu
 Classifier: Operating System :: POSIX :: Linux
 Classifier: Programming Language :: Python :: 3
 Requires-Python: >=3.12
+Requires-Dist: tomli-w
 Description-Content-Type: text/markdown
 
 ## umu-commander
@@ -48,4 +49,11 @@ umu-commander needs one of the following verbs specified after the executable na
   * This is NOT equivalent to `umu-run --config <config_name>`, as vanilla umu configs do not support setting environment variables as of 07/2025.
   
 ### Installation/Usage
-Add umu-run to your PATH and then install with pipx by running `pipx install umu-commander`. After that you can invoke umu-commander by running `umu-commander <verb>`. 
+Add umu-run to your PATH and then install with pipx by running `pipx install umu-commander`. After that you can invoke umu-commander by running `umu-commander <verb>`. 
+
+### Return codes
+| Number | Name              | Description                                                     |
+|:-------|:------------------|:----------------------------------------------------------------|
+| 0      | SUCCESS           | Program executed as intended.                                   |
+| 1      | READING_ERROR     | Failed to parse a file and could not recover.                   |
+| 2      | INVALID_SELECTION | User selected an invalid verb or there are no valid selections. |

{umu_commander-1.5.2 → umu_commander-1.5.4}/README.md

@@ -33,4 +33,11 @@ umu-commander needs one of the following verbs specified after the executable na
   * This is NOT equivalent to `umu-run --config <config_name>`, as vanilla umu configs do not support setting environment variables as of 07/2025.
   
 ### Installation/Usage
-Add umu-run to your PATH and then install with pipx by running `pipx install umu-commander`. After that you can invoke umu-commander by running `umu-commander <verb>`. 
+Add umu-run to your PATH and then install with pipx by running `pipx install umu-commander`. After that you can invoke umu-commander by running `umu-commander <verb>`. 
+
+### Return codes
+| Number | Name              | Description                                                     |
+|:-------|:------------------|:----------------------------------------------------------------|
+| 0      | SUCCESS           | Program executed as intended.                                   |
+| 1      | READING_ERROR     | Failed to parse a file and could not recover.                   |
+| 2      | INVALID_SELECTION | User selected an invalid verb or there are no valid selections. |

{umu_commander-1.5.2 → umu_commander-1.5.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "umu-commander"
-version = "1.5.2"
+version = "1.5.4"
 authors = [
   { name="Mpaxlamitsounas", email="worldstudy123@gmail.com" },
 ]
@@ -15,7 +15,7 @@ classifiers = [
     "Programming Language :: Python :: 3",
     "Operating System :: POSIX :: Linux",
 ]
-dependendies =[
+dependencies =[
     "tomli-w"
 ]
 

{umu_commander-1.5.2 → umu_commander-1.5.4}/src/umu_commander/main.py

@@ -36,7 +36,7 @@ def main() -> ExitCode:
 
     if len(sys.argv) == 1:
         print_help()
-        return ExitCode.SUCCESS
+        return ExitCode.SUCCESS.value
 
     verb: str = sys.argv[1]
     match verb:
@@ -55,14 +55,14 @@ def main() -> ExitCode:
         case _:
             print("Invalid verb.")
             print_help()
-            return ExitCode.INVALID_SELECTION
+            return ExitCode.INVALID_SELECTION.value
 
     tracking.untrack_unlinked()
     db.dump()
     config.dump()
 
-    return ExitCode.SUCCESS
+    return ExitCode.SUCCESS.value
 
 
 if __name__ == "__main__":
-    exit(main().value)
+    exit(main())

{umu_commander-1.5.2 → umu_commander-1.5.4}/src/umu_commander/util.py

@@ -58,7 +58,7 @@ def get_selection(
 ) -> Element:
     if not _selection_set_valid(selection_elements, selection_groups):
         print("Nothing to select from.")
-        exit(ExitCode.INVALID_SELECTION)
+        exit(ExitCode.INVALID_SELECTION.value)
 
     if selection_groups is None:
         selection_groups = []