fix: Remove dev_enabled and viz_enabled, use GitHub default python gi…

…tignore, remove multiline f-string

.gitignore

@@ -1,5 +1,132 @@
-*.pyc
-.DS_Store
+# Byte-compiled / optimized / DLL files
 __pycache__/
+*.py[cod]
+*$py.class
+
+# C extensions
+*.so
+
+# Distribution / packaging
+.Python
+build/
+develop-eggs/
+dist/
+downloads/
+eggs/
+.eggs/
+lib/
+lib64/
+parts/
+sdist/
+var/
+wheels/
+pip-wheel-metadata/
+share/python-wheels/
+*.egg-info/
+.installed.cfg
+*.egg
+MANIFEST
+
+# PyInstaller
+#  Usually these files are written by a python script from a template
+#  before PyInstaller builds the exe, so as to inject date/other infos into it.
+*.manifest
+*.spec
+
+# Installer logs
+pip-log.txt
+pip-delete-this-directory.txt
+
+# Unit test / coverage reports
+htmlcov/
+.tox/
+.nox/
 .coverage
-.vscode/
+.coverage.*
+.cache
+nosetests.xml
+coverage.xml
+*.cover
+*.py,cover
+.hypothesis/
+.pytest_cache/
+
+# Translations
+*.mo
+*.pot
+
+# Django stuff:
+*.log
+local_settings.py
+db.sqlite3
+db.sqlite3-journal
+
+# Flask stuff:
+instance/
+.webassets-cache
+
+# Scrapy stuff:
+.scrapy
+
+# Sphinx documentation
+docs/_build/
+
+# PyBuilder
+target/
+
+# Jupyter Notebook
+.ipynb_checkpoints
+
+# IPython
+profile_default/
+ipython_config.py
+
+# pyenv
+.python-version
+
+# pipenv
+#   According to pypa/pipenv#598, it is recommended to include Pipfile.lock in version control.
+#   However, in case of collaboration, if having platform-specific dependencies or dependencies
+#   having no cross-platform support, pipenv may install dependencies that don't work, or not
+#   install all needed dependencies.
+#Pipfile.lock
+
+# PEP 582; used by e.g. github.com/David-OConnor/pyflow
+__pypackages__/
+
+# Celery stuff
+celerybeat-schedule
+celerybeat.pid
+
+# SageMath parsed files
+*.sage.py
+
+# Environments
+.env
+.venv
+env/
+venv/
+ENV/
+env.bak/
+venv.bak/
+
+# Spyder project settings
+.spyderproject
+.spyproject
+
+# Rope project settings
+.ropeproject
+
+# mkdocs documentation
+/site
+
+# mypy
+.mypy_cache/
+.dmypy.json
+dmypy.json
+
+# Pyre type checker
+.pyre/
+
+# Visual Studio Code
+.vscode/

onair/config/default_config.ini

@@ -27,7 +27,7 @@ PlannersPluginDict = {'generic':'plugins/generic/__init__.py'}
 # Required Key: ComplexPluginDict(s) are used by Agent for complex reasoning
 ComplexPluginDict = {'generic':'plugins/generic/__init__.py'}
 
-# Required Section: OPTIONS are settable values to change running experience
+# Optional Section: OPTIONS are settable values to change running experience
 [OPTIONS]
 # Optional Key: IO_Flag denotes whether or not to provide console output
 # default = false

onair/config/reporter_config.ini

@@ -19,5 +19,3 @@ ComplexPluginDict = {'Complex Reporter 1':'plugins/reporter',
 
 [OPTIONS]
 IO_Enabled = true
-Dev_Enabled = false
-Viz_Enabled = false

onair/src/run_scripts/execution_engine.py

@@ -16,7 +16,7 @@
 import importlib
 import ast
 import shutil
-from distutils.dir_util import copy_tree
+from shutil import copytree
 from time import gmtime, strftime
 
 from ..run_scripts.sim import Simulator
@@ -65,8 +65,7 @@ def parse_configs(self, config_filepath):
         config = configparser.ConfigParser()
 
         if len(config.read(config_filepath)) == 0:
-            raise FileNotFoundError(
-                f"Config file at '{config_filepath}' could not be read.")
+            raise FileNotFoundError(f"Config file at '{config_filepath}' could not be read.")
 
         try:
             # Parse Required Data: FILES
@@ -102,8 +101,7 @@ def parse_configs(self, config_filepath):
                 self.IO_Enabled = False
 
         except KeyError as e:
-            new_message = f"Config file: '{
-                config_filepath}', missing key: {e.args[0]}"
+            new_message = f"Config file: '{config_filepath}', missing key: {e.args[0]}"
             raise KeyError(new_message) from e
 
     def parse_plugins_dict(self, config_plugin_dict):
@@ -112,13 +110,11 @@ def parse_plugins_dict(self, config_plugin_dict):
         if isinstance(ast_plugin_dict.body, ast.Dict):
             temp_plugin_dict = ast.literal_eval(config_plugin_dict)
         else:
-            raise ValueError(f"Plugin dict {config_plugin_dict} from {
-                             self.config_filepath} is invalid. It must be a dict.")
+            raise ValueError(f"Plugin dict {config_plugin_dict} from {self.config_filepath} is invalid. It must be a dict.")
 
         for plugin_file in temp_plugin_dict.values():
             if not (os.path.exists(plugin_file)):
-                raise FileNotFoundError(f"In config file '{self.config_filepath}' Plugin path '{
-                                        plugin_file}' does not exist.")
+                raise FileNotFoundError(f"In config file '{self.config_filepath}' Plugin path '{plugin_file}' does not exist.")
         return temp_plugin_dict
 
     def parse_data(self, parser_file_name, data_file_name, metadata_file_name, subsystems_breakdown=False):
@@ -171,7 +167,7 @@ def save_results(self, save_name):
         save_path = os.environ['ONAIR_SAVE_PATH'] + \
             'saved/' + save_name + '_' + complete_time
         os.makedirs(save_path, exist_ok=True)
-        copy_tree(os.environ['ONAIR_TMP_SAVE_PATH'], save_path)
+        copytree(os.environ['ONAIR_TMP_SAVE_PATH'], save_path)
 
     def set_run_param(self, name, val):
         setattr(self, name, val)