Toute premiere version de la generation auto de la doc sphinx (via pyros update)…

… : doc guitastro + doc codestyle

Toute premiere version de la generation auto de la doc sphinx (via pyros update)…
… : doc guitastro + doc codestyle
Etienne Pallier
1 parent 0318e3c9
Showing 9 changed files with 807 additions and 296 deletions Show diff stats
doc/codestyle_examples/codestyle_first.py
doc/doc_rst/.gitignore
doc/doc_rst/Makefile
doc/doc_rst/make.bat
doc/doc_rst/make_rst_then_html
doc/doc_rst/requirements.txt
doc/doc_rst/source/conf.py
doc/doc_rst/source/index.rst
pyros.py
@@ -14,6 +14,8 @@
     - Sphinx generalities on RST format : https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html
     - Sphinx autodoc : https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html
+    - Sphinx autodocsumm : https://autodocsumm.readthedocs.io/en/latest/index.html
+    - Sphinx autosummary : https://www.sphinx-doc.org/en/master/usage/extensions/autosummary.html
     - Sphinx apidoc (1) for autogeneration of files in the source folder : https://www.sphinx-doc.org/en/master/man/sphinx-apidoc.html
     - Sphinx apidoc (2) more explanations : https://samnicholls.net/2016/06/15/how-to-sphinx-readthedocs
     - Sphinx inheritance diagrams : https://www.sphinx-doc.org/en/master/usage/extensions/inheritance.html
@@ -113,8 +115,6 @@ def general_function_that_returns_a_float(
         general_function_that_returns_a_float(arg_a=1, arg_b="toto")    # => OK
         general_function_that_returns_a_float(arg_a=1, arg_b=1.2)       # => KO (float not allowed)
-
-
     """
     # comment on a
@@ -0,0 +1,3 @@
+build*/*
+TMP
+source/generated_api/*
@@ -0,0 +1,20 @@
+# Minimal makefile for Sphinx documentation
+#
+
+# You can set these variables from the command line, and also
+# from the environment for the first two.
+SPHINXOPTS    ?=
+SPHINXBUILD   ?= sphinx-build
+SOURCEDIR     = source
+BUILDDIR      = build
+
+# Put it first so that "make" without argument is like "make help".
+help:
+	@$(SPHINXBUILD) -M help "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
+
+.PHONY: help Makefile
+
+# Catch-all target: route all unknown targets to Sphinx using the new
+# "make mode" option.  $(O) is meant as a shortcut for $(SPHINXOPTS).
+%: Makefile
+	@$(SPHINXBUILD) -M $@ "$(SOURCEDIR)" "$(BUILDDIR)" $(SPHINXOPTS) $(O)
@@ -0,0 +1,35 @@
+@ECHO OFF
+
+pushd %~dp0
+
+REM Command file for Sphinx documentation
+
+if "%SPHINXBUILD%" == "" (
+	set SPHINXBUILD=sphinx-build
+)
+set SOURCEDIR=source
+set BUILDDIR=build
+
+if "%1" == "" goto help
+
+%SPHINXBUILD% >NUL 2>NUL
+if errorlevel 9009 (
+	echo.
+	echo.The 'sphinx-build' command was not found. Make sure you have Sphinx
+	echo.installed, then set the SPHINXBUILD environment variable to point
+	echo.to the full path of the 'sphinx-build' executable. Alternatively you
+	echo.may add the Sphinx directory to PATH.
+	echo.
+	echo.If you don't have Sphinx installed, grab it from
+	echo.http://sphinx-doc.org/
+	exit /b 1
+)
+
+%SPHINXBUILD% -M %1 %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%
+goto end
+
+:help
+%SPHINXBUILD% -M help %SOURCEDIR% %BUILDDIR% %SPHINXOPTS% %O%
+
+:end
+popd
@@ -0,0 +1,44 @@
+
+# A - Generate RST files
+# ---------------------------------
+
+#SPHINX_APIDOC_OPTIONS=autosummary,autosummary-no-nesting,members,undoc-members,show-inheritance
+##export SPHINX_APIDOC_OPTIONS=members,undoc-members,show-inheritance,autosummary,autosummary-no-nesting
+
+#sphinx-apidoc -f -o source ../../src/core
+
+# 1) guitastro package (celme)
+# Doc au format de Alain
+cd ../../vendor/guitastro/doc_rst/
+pwd
+make html
+# Doc au format de Etienne (just uncomment 1st line)
+###sphinx-apidoc -f -o source/generated_api -a --separate --tocfile guitastro_package ../../vendor/guitastro
+###sphinx-apidoc -f -o source/generated_api -a --separate --module-first --tocfile guitastro_package ../../vendor/guitastro
+#sphinx-apidoc -f -o source -a --separate --module-first --tocfile guitastro_package ../../vendor/guitastro
+#sphinx-apidoc -f -o source -a --separate --module-first --tocfile celme_package ../../src/core/celme
+#sphinx-apidoc -o source --tocfile celme_package ../../src/core/celme
+cd -
+pwd
+
+# 2) device_controller package
+######sphinx-apidoc -o source -a --separate --module-first --tocfile device_controller_package ../../src/device_controller
+
+# 3) pyros_django package
+####sphinx-apidoc -f -o source -a --separate --module-first --tocfile pyros_django_package ../../src/core/pyros_django
+#sphinx-apidoc -f -o source --tocfile pyros_django_package ../../src/core/pyros_django
+
+# 4) Code style package
+sphinx-apidoc -f -o source/generated_api -a --separate --module-first --tocfile codestyle_examples_package ../codestyle_examples
+##sphinx-apidoc -f -o source -a --separate --module-first --tocfile codestyle_examples_package ../codestyle_examples
+#sphinx-apidoc -o source -a --separate --module-first --tocfile code_style_package ../code_style/my_package1
+
+#sphinx-apidoc -f -o source ../../src
+#sphinx-apidoc -f -o --implicit-namespaces source ../../src/
+
+
+# B - Generate HTML (and pdf) files (from RST)
+# ---------------------------------
+
+sphinx-build -b html source build/html/
+#sphinx-build -b pdf source build
@@ -0,0 +1,15 @@
+Sphinx
+sphinx-autodoc-typehints
+sphinx-pyreverse
+
+# Extensions
+autodocsumm
+
+# Themes
+#alabaster
+sphinx-rtd-theme
+
+# Builder
+rst2pdf
+#rstcheck
+
@@ -0,0 +1,214 @@
+# Configuration file for the Sphinx documentation builder.
+#
+# This file only contains a selection of the most common options. For a full
+# list see the documentation:
+# https://www.sphinx-doc.org/en/master/usage/configuration.html
+
+# -- Path setup --------------------------------------------------------------
+
+# If extensions (or modules to document with autodoc) are in another directory,
+# add these directories to sys.path here. If the directory is relative to the
+# documentation root, use os.path.abspath to make it absolute, like shown here.
+#
+import os
+import sys
+# sys.path.insert(0, os.path.abspath('.'))
+
+#import pathlib
+#sys.path.insert(0, pathlib.Path(__file__).parents[2].resolve().as_posix())
+
+# - for code style examples :
+sys.path.insert(0, os.path.abspath('../../../doc'))
+# - for PyROS :
+sys.path.insert(0, os.path.abspath('../../..'))
+sys.path.insert(0, os.path.abspath('../../../src'))
+sys.path.insert(0, os.path.abspath('../../../src/core'))
+# - for guitastro :
+sys.path.insert(0, os.path.abspath('../../../vendor'))
+#sys.path.insert(0, os.path.abspath('../../../src/device_controller'))
+#sys.path.insert(0, os.path.abspath('../../../src/core/pyros_django'))
+#sys.path.insert(0, os.path.abspath('../../../src/core/celme'))
+sys.setrecursionlimit(1500)
+
+
+# -- Project information -----------------------------------------------------
+
+project = 'PyROS'
+copyright = '2022, E. Pallier, A. Klotz, A. Koralewski'
+author = 'E. Pallier, A. Klotz, A. Koralewski'
+
+# The full version, including alpha/beta/rc tags
+release = '0.4.1'
+
+
+# -- General configuration ---------------------------------------------------
+
+# Add any Sphinx extension module names here, as strings. They can be
+# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom
+# ones.
+
+extensions = [
+    'sphinx.ext.autodoc',  # automatically generate documentation for modules
+
+    # (EP) sphinx-autodoc-typehints
+    # https://pypi.org/project/sphinx-autodoc-typehints
+    # TODO: pip install sphinx-autodoc-typehints
+    # TODO: make sure you load sphinx.ext.napoleon first, before sphinx-autodoc-typehints
+    'sphinx.ext.napoleon',  # to read Google-style or Numpy-style docstrings
+    'sphinx_autodoc_typehints',
+
+    'sphinx.ext.todo',
+
+    #'sphinx.ext.duration',
+    'sphinx.ext.doctest',
+
+    #'sphinx.ext.autosummary',
+    "autodocsumm",  # to generate tables of functions, attributes, methods, etc. (python3 -m pip install autodocsumm)
+    'sphinx.ext.intersphinx',
+    'sphinx.ext.mathjax',
+    'sphinx.ext.viewcode',  # to allow vieing the source code in the web page
+    'sphinx.ext.graphviz',
+    #'sphinx_pyreverse',
+    'rst2pdf.pdfbuilder',
+    # to be used with option : show-inheritance-diagram (mais ne marche pas pour l'instant...)
+    'sphinx.ext.inheritance_diagram',
+]
+
+'''
+try:
+    extensions.append('sphinx_pyreverse')
+except ModuleNotFoundError:
+    pass # pip install sphinx_pyreverse
+
+try:
+    import rst2pdf
+    extensions.append('rst2pdf.pdfbuilder')
+except ModuleNotFoundError:
+    pass # no rst2pdf for you
+'''
+
+# (EP) To avoid sorting of methods & attributes, very annoying... (so, keep them in samed order as in source)
+#autodoc_member_order = 'bysource'
+
+# (EP) Default config for autodoc
+#'members': 'var1, var2',
+#'special-members': '__init__, __str__',
+#'special-members': '__str__',
+#'autoclass_content': 'class',
+#'exclude-members': '__init__, __weakref__',
+autodoc_default_options = {
+
+     # make autodocsumm active by default for all autodoc directives
+    'autosummary': True,
+
+    # autodoc will also generate document for the undoc-umented members (not having docstrings)
+    'undoc-members': True,
+
+    'member-order': 'bysource',
+    'special-members': '__str__',
+    #'special-members': '__init__, __str__',
+    #'exclude-members': '__weakref__',
+    'show-inheritance': True,
+    #'show-inheritance': False,
+    # ne marche pas ?
+    'show-inheritance-diagram': True,
+
+    'typehints_fully_qualified': False,
+
+    # (EP) nothing below is taken into account...
+    ##'add_module_names': False,
+    ##'autoclass_content': 'init',
+    #'autoclass_content': 'both',
+    #'autodoc_typehints': 'signature',
+    ##'autodoc_typehints': 'both',
+    #'autodoc_typehints': 'description',
+    ##'autodoc_typehints_description_target': 'all',
+    #'autodoc_class_signature': 'mixed',
+    #'autodoc_class_signature': 'separated',
+}
+#print(autodoc_default_options)
+#exit
+
+# (EP) necessary because not taken into account in autodoc_default_options above !!!!!!
+
+
+# - short module name
+add_module_names = False
+
+# Le commentaire de la methode __init__ est rajouté au commentaire de la classe ?
+# https://www.sphinx-doc.org/en/master/usage/extensions/autodoc.html#confval-autoclass_content
+# - NON
+#autoclass_content = 'class'
+# - OUI
+#autoclass_content = 'init'
+# - OUI, les 2 commentaires (class et init) sont fusionnés
+autoclass_content = 'both'
+
+# - Documenter automatiquement TOUS les arguments de methodes, meme ceux qui ne sont pas explicitement documentés
+#autodoc_typehints = 'both'
+#autodoc_typehints = 'signature'
+# Show types only in descriptions, not in signatures
+autodoc_typehints = "description"
+
+# don't include docstrings from the parent class
+autodoc_inherit_docstrings = False
+
+# This value controls whether the types of undocumented parameters and return values are documented when autodoc_typehints is set to description.
+# The default value is "all", meaning that types are documented for all parameters and return values, whether they are documented or not.
+# When set to "documented", types will only be documented for a parameter or a return value that is already documented by the docstring.
+##autodoc_typehints_description_target = 'all'
+
+# Afficher le nom de la classe AVEC ses arguments ou SANS ?
+# - AVEC
+autodoc_class_signature = 'mixed'
+# - SANS
+#autodoc_class_signature = 'separated'
+
+
+# Napoleon settings
+# https://www.sphinx-doc.org/en/master/usage/extensions/napoleon.html#configuration
+napoleon_google_docstring = True
+napoleon_numpy_docstring = False
+#napoleon_include_init_with_doc = False
+'''
+napoleon_include_private_with_doc = False
+napoleon_include_special_with_doc = True
+napoleon_use_admonition_for_examples = False
+napoleon_use_admonition_for_notes = False
+napoleon_use_admonition_for_references = False
+napoleon_use_ivar = False
+napoleon_use_param = True
+napoleon_use_rtype = True
+napoleon_preprocess_types = False
+napoleon_type_aliases = None
+napoleon_attr_annotations = True
+'''
+
+# Add any paths that contain templates here, relative to this directory.
+templates_path = ['_templates']
+
+# List of patterns, relative to source directory, that match files and
+# directories to ignore when looking for source files.
+# This pattern also affects html_static_path and html_extra_path.
+#exclude_patterns = []
+exclude_patterns = ['_build', 'Thumbs.db', '.DS_Store']
+
+
+# -- Options for HTML output -------------------------------------------------
+
+# The theme to use for HTML and HTML Help pages.  See the documentation for
+# a list of builtin themes.
+#
+#html_theme = 'alabaster'
+# (EP) Read The Docs theme
+html_theme = 'sphinx_rtd_theme'
+# Set html_theme to default for “standard” ReadTheDocs theme
+#html_theme = 'default'
+#html_theme = 'nature'
+
+# Add any paths that contain custom static files (such as style sheets) here,
+# relative to this directory. They are copied after the builtin static files,
+# so a file named "default.css" will overwrite the builtin "default.css".
+html_static_path = ['_static']
+
+autosummary_generate = True
@@ -0,0 +1,97 @@
+*****************************************************
+Welcome in the PyROS developer documentation
+*****************************************************
+
+.. image:: ./doc_images/logo-irap-couleur-1024x386.png
+	:height: 100px
+	:align: center
+
++------------------------------+------------------------------------------------------------------+
+| | IRAP                       | | Project Scientist (PI) : Alain Klotz - aklotz AT irap.omp.eu   |
+| | 9 Avenue Colonel Roche     | | Project Manager : Etienne Pallier - epallier AT irap.omp.eu    |
+| | 31028 Toulouse Cedex 4     | | Developper : Alexis Koralewski - akoralewski AT irap.omp.eu    |
+| | FRANCE                     | | http://pyros.irap.omp.eu                                       |
++------------------------------+------------------------------------------------------------------+
+
+.. warning:: This is a warning message, if needed
+
+
+
+
+
+1. PyROS files
+**************
+
+See in the Installation documentation - Project structure <https://docs.google.com/document/d/1DjT_rAzeVPl9SnQG3_20GFbDf67JEUAsOD1uhDMlOVw/edit#heading=h.19nwnkgoi5k>`_
+
+.. comments
+    .. toctree:: 
+       :maxdepth: 3
+        
+       using_mount_classes
+
+2. PyROS configuration
+**********************
+
+See in the Design&Implementation documentation - PyROS configuration <https://docs.google.com/document/d/1osHT3d8GiofRd9k3nL7cz4Hcmo9WVEOcSaSSQpgJcFs/edit#heading=h.21n0fso0r8u9>`_
+
+
+
+3. Observatory configuration
+****************************
+
+See in the Design&Implementation documentation - Observatory configuration <https://docs.google.com/document/d/1osHT3d8GiofRd9k3nL7cz4Hcmo9WVEOcSaSSQpgJcFs/edit#heading=h.ps1odi23szui>`_
+
+
+
+4. PyROS API Source Code documentation
+**************************************
+
+The PyROS Packages, Classes, and Methods
+
+
+
+5. Guitastro doc
+****************
+
+`Guitastro documentation <../../../../vendor/guitastro/doc_rst/build/html/index.html>`_
+
+
+
+
+
+
+.. COMMENT: The toctree directive is mainly to let Sphinx be aware of the document hierarchy (the sidebar will reflect this understanding)
+
+
+.. toctree:: 
+    :maxdepth: 3
+    :caption: Contents:
+
+    generated_api/codestyle_examples_package
+
+.. comment    guitastro
+
+.. comment    `Guitastro documentation <../../../vendor/guitastro/doc_rst/build/html/index.html>`_
+
+.. comment    ../../../vendor/guitastro/doc_rst/source/index
+
+.. Generation doc guitastro API directement depuis ce script (EP) :
+..    generated_api/guitastro_package
+
+.. Autres modules :
+..   device_controller_package
+..   pyros_django_package
+..   autodoc_pyros
+..   pyros-doc-DEV
+..   modules
+
+
+Indices and tables
+==================
+
+* :ref:`genindex`
+* :ref:`modindex`
+
+.. * :ref:`search`
+
@@ -9,24 +9,11 @@
 ''' *** STANDARD IMPORTS *** '''
-import argparse
-import fileinput
-import glob
-import os
-import platform
-import shutil
-import signal
-import subprocess
-import sys
-import time
-import re
-import glob
 #import logging
 ''' *** PROJECT IMPORTS *** '''
 #from src.pyros_logger import *
-from src.pyros_logger import log
 #from src.pyros_logger import log, log_d,log_i,log_w,log_e,log_c
 #from src import pyros_logger
@@ -39,6 +26,19 @@ from src.pyros_logger import log
 #DEBUG = False
+import argparse
+import fileinput
+import glob
+import os
+import platform
+import shutil
+import signal
+import subprocess
+import sys
+import time
+import re
+import glob
+from src.pyros_logger import log
 UPDATE = False
 PYROS_DJANGO_BASE_DIR = "src/core/pyros_django"
@@ -47,26 +47,26 @@ INIT_FIXTURE = &quot;initial_fixture_TZ.json&quot;
 _previous_dir = None
 AGENTS = {
-    #"agentX" : "agent", 
-    "agent" : "Agent", 
-    "agent2" : "Agent", 
-    "agentX" : "AgentX", 
-    "agentA" : "AgentA", 
-    "agentB" : "AgentB", 
-    "agentC" : "AgentC", 
-    "agentM" : "AgentM",
-    "agentSP" : "AgentSP", 
-    #"agentDevice" : "AgentDevice", 
-    #"agentDeviceTelescopeGemini" : "AgentDeviceTelescopeGemini",
-    "agentDeviceGemini" : "AgentDeviceGemini",
-    "agentDeviceSBIG" : "AgentDeviceSBIG",
-    "agentTelescopeRequester" : "AgentTelescopeRequester", 
-    "agentMultiRequester" : "AgentMultiRequester", 
-    "webserver" : "webserver", 
-    "monitoring" : "monitoring", 
-    "majordome" : "majordome", 
-    "scheduler" : "scheduler", 
-    "alert_manager" : "alert_manager"
+    # "agentX" : "agent",
+    "agent": "Agent",
+    "agent2": "Agent",
+    "agentX": "AgentX",
+    "agentA": "AgentA",
+    "agentB": "AgentB",
+    "agentC": "AgentC",
+    "agentM": "AgentM",
+    "agentSP": "AgentSP",
+    # "agentDevice" : "AgentDevice",
+    # "agentDeviceTelescopeGemini" : "AgentDeviceTelescopeGemini",
+    "agentDeviceGemini": "AgentDeviceGemini",
+    "agentDeviceSBIG": "AgentDeviceSBIG",
+    "agentTelescopeRequester": "AgentTelescopeRequester",
+    "agentMultiRequester": "AgentMultiRequester",
+    "webserver": "webserver",
+    "monitoring": "monitoring",
+    "majordome": "majordome",
+    "scheduler": "scheduler",
+    "alert_manager": "alert_manager"
 }
 #AGENTS = ["agentX", "webserver", "monitoring", "majordome", "scheduler", "alert_manager"]
 #AGENTS = ["all", "webserver", "monitoring", "majordome", "scheduler", "alert"]
@@ -86,23 +86,23 @@ if IS_WINDOWS:
     PYTHON = "python.exe"
-try :
+try:
     # WITH_DOCKER is an environment varialbe from our Docker image
-    WITH_DOCKER=os.environ['WITH_DOCKER']
+    WITH_DOCKER = os.environ['WITH_DOCKER']
 except KeyError:
-    WITH_DOCKER=False
+    WITH_DOCKER = False
-if type(WITH_DOCKER) is str and re.match("^y$|^Y$|^yes$|^Yes$",WITH_DOCKER.rstrip()) != None : 
-    WITH_DOCKER = True 
-else : 
+if type(WITH_DOCKER) is str and re.match("^y$|^Y$|^yes$|^Yes$", WITH_DOCKER.rstrip()) != None:
+    WITH_DOCKER = True
+else:
     WITH_DOCKER = False
 my_abs_path = os.path.dirname(os.path.realpath(__file__))
 #VENV_ROOT = "private"
-if WITH_DOCKER : 
+if WITH_DOCKER:
     VENV_ROOT = ""
     VENV = ""
-    VENV_BIN =  ""
+    VENV_BIN = ""
 else:
     VENV_ROOT = "venv"
     VENV = "venv_py3_pyros"
@@ -116,6 +116,7 @@ else:
 VENV_PYTHON = VENV_BIN + PYTHON
 VENV_PIP = VENV_BIN + 'pip'
+
 class Colors:
     HEADER = "\033[95m"
     BLUE = "\033[94m"
@@ -134,9 +135,6 @@ class Colors:
         LOG_BLUE = ''
-
-
-
 '''
 VARIABLES FOR INSTALL
 '''
@@ -148,7 +146,7 @@ INSTALL_DB = True
 SQL_DATABASE = "pyros"
 # Database automatically created by Django for tests (and automatically deleted after tests)
 SQL_DATABASE_TEST = "test_pyros"
-# Specific database that we need(ed?) for some simulations 
+# Specific database that we need(ed?) for some simulations
 SQL_DATABASE_SIMU = "pyros_test"
 SQL_USER = ""
 SQL_PSWD = ""
@@ -158,8 +156,6 @@ ENV_PATH = &quot;docker/variables.env&quot;
 ENV_SAMPLE_PATH = "docker/.env-sample"
-
-
 END_OF_LINE = '\n\n'
 ##VENV_BIN = '/bin/'
 # --------------------------------------------
@@ -171,7 +167,7 @@ if (platform.system() == &quot;Windows&quot;):
     #MYSQL_EXE_PATH = "C:/Program Files (x86)/MySQL/MySQL Server 5.0/bin/"
     #question = "Enter the path of the MySQL server if it is not the following name (" + MYSQL_EXE_PATH + "): "
     #res = input(question)
-    #if res!="":
+    # if res!="":
     #MYSQL_EXE_PATH = res
 #VENV_PIP = VENV + VENV_BIN+'pip'
@@ -179,16 +175,11 @@ if (platform.system() == &quot;Windows&quot;):
 # GLOBAL_PYTHON = 'python3'
 GLOBAL_PYTHON = os.path.split(sys.executable)[-1]
-log.debug(Colors.LOG_BLUE + "Python executable is " + GLOBAL_PYTHON + Colors.END)
+log.debug(Colors.LOG_BLUE + "Python executable is " +
+          GLOBAL_PYTHON + Colors.END)
 ##if platform.dist()[0] == "centos": print("centos platform")
-
-
-
-
-
-
 """
 *************************************************************************************************
 ******************** click package install (if not yet done) and import it **********************
@@ -197,7 +188,7 @@ log.debug(Colors.LOG_BLUE + &quot;Python executable is &quot; + GLOBAL_PYTHON + Colors.END
 # First thing first, install the click package !!!
 #import fire
 try:
-    import click # https://click.palletsprojects.com
+    import click  # https://click.palletsprojects.com
 except:
     #pip = "pip" if platform.system() == "Windows" else "pip3"
     # TODO: "python -m pip" au lieu de "pip"
@@ -211,10 +202,6 @@ except:
     else:
         log.error("click package installation failed")
         # self.addError(self.current_command, command)
-        
-    
-
-
 """
@@ -223,6 +210,7 @@ except:
 **************************************************************************
 """
+
 def _in_dir(dirname: str = ""):
     return os.path.basename(os.getcwd()) == dirname
@@ -232,22 +220,23 @@ def _in_abs_dir(dirname: str = &quot;&quot;):
 def die(msg: str = ""):
-    log.error()
+    log.error('')
     log.error("...ERROR...")
     log.error(msg)
-    log.error()
+    log.error('')
     exit(1)
-    
-#TODO: implement is_async
+
+# TODO: implement is_async
 def execProcess(command, from_venv=False, is_async=False):
     from_venv_str = " from venv ("+VENV_PYTHON+")" if from_venv else ""
     #printFullTerm(Colors.BLUE, "Executing command" + " [" + command + "]" + from_venv_str)
     log.debug("Executing command" + " [" + command + "]" + from_venv_str)
-    if from_venv: command = VENV_PYTHON+' ' + command
+    if from_venv:
+        command = VENV_PYTHON+' ' + command
     process = subprocess.Popen(command, shell=True)
     if is_async:
-        #current_processes.append(process)
+        # current_processes.append(process)
         return process
     # Not is_async => Wait for end of execution
     process.wait()
@@ -257,15 +246,18 @@ def execProcess(command, from_venv=False, is_async=False):
     else:
         printFullTerm(Colors.WARNING, "Process execution failed")
         # self.addError(self.current_command, command)
-    #return process.returncode
-    return True if process.returncode==0 else False
+    # return process.returncode
+    return True if process.returncode == 0 else False
+
-def execProcessFromVenv(command:str, is_async=False):
-    #return execProcess(command, from_venv=True, is_async)
+def execProcessFromVenv(command: str, is_async=False):
+    # return execProcess(command, from_venv=True, is_async)
     return execProcess(command, True, is_async)
-#TODO: fusionner dans execProcess avec param is_async
-def execProcessFromVenvAsync(command:str):
+# TODO: fusionner dans execProcess avec param is_async
+
+
+def execProcessFromVenvAsync(command: str):
     return execProcessFromVenv(command, True)
     """
     args = command.split()
@@ -280,13 +272,14 @@ def execProcessFromVenvAsync(command:str):
     return p
     """
+
 def printColor(color: Colors, message, file=sys.stdout, eol=os.linesep, forced=False):
     #system = platform.system()
     """
     if (self.disp == False and forced == False):
         return 0
     """
-    #if system == "Windows":
+    # if system == "Windows":
     if IS_WINDOWS:
         print(message, file=file, end=eol)
     else:
@@ -306,7 +299,8 @@ def printFullTerm(color: Colors, string: str):
     printColor(color, "-" * (columns - value))
     return 0
-def set_environment_variables_if_not_configured(env_path:str, env_sample_path:str)->None:
+
+def set_environment_variables_if_not_configured(env_path: str, env_sample_path: str) -> None:
     """
     Set environment variables if they aren't defined in the current environment.
     Get the variables from .env file if it exists or create this file using a copy of the env_sample
@@ -314,7 +308,7 @@ def set_environment_variables_if_not_configured(env_path:str, env_sample_path:st
     Args:
         env_path (str): path to .env file
         env_sample_path (str): path to .env-sample file
-    """   
+    """
     """
     variables_names = ["MYSQL_ROOT_PASSWORD","MYSQL_ROOT_LOGIN","MYSQL_TCP_PORT","MYSQL_PYROS_LOGIN","MYSQL_PYROS_PWD","PATH_TO_OBSCONF_FILE"]
     is_environment_variables_defined = True    
@@ -328,7 +322,7 @@ def set_environment_variables_if_not_configured(env_path:str, env_sample_path:st
     """
     log.debug("Start Setting environment variables")
     try:
-        with open(env_path,"r") as env_file:
+        with open(env_path, "r") as env_file:
             # env file is empty
             if os.stat(env_path).st_size == 0:
                 raise BaseException()
@@ -338,41 +332,43 @@ def set_environment_variables_if_not_configured(env_path:str, env_sample_path:st
                     continue
                 else:
                     line = line.rstrip()
-                    key,value = line.split("=")
+                    key, value = line.split("=")
                     # setting variables as environment variables
                     if WITH_DOCKER and os.environ.get(key) != value:
-                        log.warning(f"WARNING: Environment value for '{key}' needs to be uptaded (To remove this message : restart docker container to update them).'")
+                        log.warning(
+                            f"WARNING: Environment value for '{key}' needs to be uptaded (To remove this message : restart docker container to update them).'")
                     else:
                         # if not WITH_DOCKER and os.environ.get(key) is None:
-                        #     log.error(f"ERROR: Error while reading value for '{key}'.Please fix the '{env_file}' file. For this run, PyROS will take the values from '{env_path}'")    
+                        #     log.error(f"ERROR: Error while reading value for '{key}'.Please fix the '{env_file}' file. For this run, PyROS will take the values from '{env_path}'")
                         #     raise BaseException()
                         if WITH_DOCKER and os.environ.get(key) is None:
-                            log.warning(f"WARNING: Environment value for '{key}' isn't defined within the container (To remove this message : restart docker container to update environment values). PyROS will take the values from '{env_path}'")    
+                            log.warning(
+                                f"WARNING: Environment value for '{key}' isn't defined within the container (To remove this message : restart docker container to update environment values). PyROS will take the values from '{env_path}'")
                             raise BaseException()
                         os.environ[key] = value
     except:
         #print(f".env not found at {env_path} or is empty, creating a file at this path from the .env-sample file stored at {ENV_SAMPLE_PATH}\nvalues from .env-sample will be used as environment variables")
-        with open(env_sample_path,'r') as env_sample_file:
-            with open(env_path,"w") as env_file:
+        with open(env_sample_path, 'r') as env_sample_file:
+            with open(env_path, "w") as env_file:
                 for env_sample_line in env_sample_file:
                     if(env_sample_line.startswith("#") or not env_sample_line.strip()):
                         continue
-                    key,value = env_sample_line.split("=")
+                    key, value = env_sample_line.split("=")
                     os.environ[key] = value
                     env_file.write(env_sample_line)
     # PYROS_DJANGO_BASE_DIR isn't an absolute path so we need to add the path to current folder to it
-    os.environ["DJANGO_PATH"] = os.path.join(os.getcwd(),PYROS_DJANGO_BASE_DIR)
+    os.environ["DJANGO_PATH"] = os.path.join(
+        os.getcwd(), PYROS_DJANGO_BASE_DIR)
     log.debug("End Setting environment variables")
+
 """
 else:
     print("The environment variables are already configured, skipping this step...")
 """
-
-
 """
 ********************************************************************************
 ******************** CLI COMMANDS DEFINITION (click format) ********************
@@ -404,21 +400,22 @@ def test_mode(): return GLOBAL_OPTIONS[&quot;test&quot;]
 def sim_mode(): return GLOBAL_OPTIONS["sim"]
 def debug_mode(): return GLOBAL_OPTIONS["debug"]
+
 @click.group()
 @click.option('--debug', '-d', is_flag=True, help='DEBUG mode')
 @click.option('--sim', '-s', is_flag=True, help="only for an AgentDevice, asking it to start its simulator and work with it (instead of real device)")
 @click.option('--test', '-t', is_flag=True, help="don't do it for real, just show what it would do")
 @click.option('--verbose', '-v', is_flag=True, help='Verbose output')
-#@click.option('--verbose', '-v', 'verbosity', flag_value=2, default=1, help='Verbose output'),
-#@click.option('--quiet', '-q', 'verbosity', flag_value=0, help='Minimal output'),
-#@click.option('--fail-fast', '--failfast', '-f', 'fail_fast', is_flag=True, default=False, help='Stop on failure'),
+# @click.option('--verbose', '-v', 'verbosity', flag_value=2, default=1, help='Verbose output'),
+# @click.option('--quiet', '-q', 'verbosity', flag_value=0, help='Minimal output'),
+# @click.option('--fail-fast', '--failfast', '-f', 'fail_fast', is_flag=True, default=False, help='Stop on failure'),
 def pyros_launcher(debug, sim, test, verbose):
     ##global log
     #log.info("in pyros launcher")
-    #pass
+    # pass
     os.environ['PYROS_DEBUG'] = '1' if debug else '0'
-    
-    ##pyros_logger.set_logger_config()
+
+    # pyros_logger.set_logger_config()
     ##log = logging.getLogger('pyroslogger')
     log.debug('starting pyros')
     #log.info('info toto msg from pyros')
@@ -426,21 +423,22 @@ def pyros_launcher(debug, sim, test, verbose):
     #log.error('error toto msg from pyros')
     #log.critical('error toto msg from pyros')
-    if debug: click.echo('DEBUG mode')
-    if sim: click.echo('Starting and connecting to simulator instead of real device')
-    if test: click.echo('Test mode')
-    if verbose: click.echo('Verbose mode')
+    if debug:
+        click.echo('DEBUG mode')
+    if sim:
+        click.echo('Starting and connecting to simulator instead of real device')
+    if test:
+        click.echo('Test mode')
+    if verbose:
+        click.echo('Verbose mode')
     GLOBAL_OPTIONS["debug"] = debug
     GLOBAL_OPTIONS["sim"] = sim
     GLOBAL_OPTIONS["test"] = test
     GLOBAL_OPTIONS["verbose"] = verbose
-
-
-
 @pyros_launcher.command(help="Run a pyros shell (django included)")
-#@global_test_options
+# @global_test_options
 def shell():
     print("Execution commande shell")
     print("NAME IS", __name__)
@@ -473,9 +471,10 @@ def shell():
     change_dir(PYROS_DJANGO_BASE_DIR)
     # execProcess("python install.py install")
-    if not test_mode(): execProcessFromVenv("manage.py shell")
+    if not test_mode():
+        execProcessFromVenv("manage.py shell")
     # Go back to the initial dir
-    #os.chdir("../")
+    # os.chdir("../")
     change_dir("PREVIOUS")
     return True
@@ -489,7 +488,8 @@ def dbshell():
     print("Type 'exit' to quit")
     print()
     # execProcess("python install.py install")
-    if not test_mode(): execProcessFromVenv(PYROS_DJANGO_BASE_DIR+"/manage.py dbshell")
+    if not test_mode():
+        execProcessFromVenv(PYROS_DJANGO_BASE_DIR+"/manage.py dbshell")
     # Go back to the initial dir
     return True
@@ -497,22 +497,27 @@ def dbshell():
 @pyros_launcher.command(help="Install the pyros software")
 @click.option('--packages_only', '-p', is_flag=True, help='install only the python packages (no database installation)')
 @click.option('--database_only', '-d', is_flag=True, help='install only the pyros database (no python packages installation)')
-#@global_test_options
+# @global_test_options
 def install(packages_only, database_only):
-    install_or_update(UPDATE=False, packages_only=packages_only, database_only=database_only)
+    install_or_update(UPDATE=False, packages_only=packages_only,
+                      database_only=database_only)
-def install_or_update(UPDATE:bool=False, packages_only:bool=False, database_only:bool=False):
+
+def install_or_update(UPDATE: bool = False, packages_only: bool = False, database_only: bool = False):
     SQL_USER = os.environ.get("MYSQL_PYROS_LOGIN").strip()
     SQL_PSWD = os.environ.get("MYSQL_PYROS_PWD").strip()
-    os.environ["PATH_TO_OBSCONF_FOLDER"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../privatedev/config/default/")
-    ACTION = "UPDATING" if UPDATE else "INSTALLING" 
+    os.environ["PATH_TO_OBSCONF_FOLDER"] = os.path.join(os.path.abspath(
+        PYROS_DJANGO_BASE_DIR), "../../../privatedev/config/default/")
+    ACTION = "UPDATING" if UPDATE else "INSTALLING"
+
     if WITH_DOCKER:
         database_only = True
     if not packages_only and not database_only:
         packages_only = database_only = True
-    if UPDATE: 
+
+    if UPDATE:
         print("Running UPDATE command")
-    else: 
+    else:
         print("Running INSTALL command")
         print("- packages_only:", packages_only)
         print("- database_only:", database_only)
@@ -521,17 +526,20 @@ def install_or_update(UPDATE:bool=False, packages_only:bool=False, database_only
             from git import Repo
         except:
             pip = "pip" if IS_WINDOWS else "pip3"
-            process = subprocess.Popen(pip + " install --upgrade GitPython", shell=True)
+            process = subprocess.Popen(
+                pip + " install --upgrade GitPython", shell=True)
             process.wait()
             if process.returncode == 0:
-            # self.addExecuted(self.current_command, command)
+                # self.addExecuted(self.current_command, command)
                 from git import Repo
             else:
-                log.error("GitPython package (required for obsconfig class) installation failed")
+                log.error(
+                    "GitPython package (required for obsconfig class) installation failed")
-        if not os.path.exists( os.path.join(os.getcwd(),"./vendor/guitastro")):
+        if not os.path.exists(os.path.join(os.getcwd(), "./vendor/guitastro")):
             print("Cloning Guiastro repository")
-            cloned_repo = Repo.clone_from("https://gitlab.irap.omp.eu/aklotz/guitastro.git", os.path.join(os.getcwd(),"./vendor/guitastro"))
+            cloned_repo = Repo.clone_from(
+                "https://gitlab.irap.omp.eu/aklotz/guitastro.git", os.path.join(os.getcwd(), "./vendor/guitastro"))
             print("Cloned successfully: ", cloned_repo.__class__ is Repo)
     #if test_mode(): print("in test mode")
     # self.execProcess("python3 install/install.py install")
@@ -540,63 +548,75 @@ def install_or_update(UPDATE:bool=False, packages_only:bool=False, database_only
     num = 0
     # 1) Update source code (git pull)
     if UPDATE:
-        num+=1
-        printFullTerm(Colors.BLUE, f"{num}) UPDATING SOURCE CODE: Running git pull")
+        num += 1
+        printFullTerm(
+            Colors.BLUE, f"{num}) UPDATING SOURCE CODE: Running git pull")
         if not WITH_DOCKER:
             _gitpull() or die()
     # 2) Update python packages (pip upgrade AND pip install requirements)
     if packages_only:
-        num+=1
+        num += 1
         printFullTerm(Colors.BLUE, f"{num}) {ACTION} PYTHON PACKAGES")
         # (UPDATE) Re-install VENV if disappeared
-        install_venv(EVEN_IF_ALREADY_EXISTS = not UPDATE)
+        install_venv(EVEN_IF_ALREADY_EXISTS=not UPDATE)
         install_packages()
     # 3) Update PlantUML diagrams
-    num+=1
+    num += 1
     printFullTerm(Colors.BLUE, f"{num}) UPDATING UML DIAGRAMS")
     _update_plantuml_diags() or die()
     print(os.getcwd())
-    
-    # 4) Install/Update database structure (make migrations + migrate)
-    if database_only: 
-        num+=1
+
+    # 4) Update Sphinx API doc
+    num += 1
+    printFullTerm(Colors.BLUE, f"{num}) UPDATING API DOC (with Sphinx)")
+    _update_api_doc() or die()
+    exit()
+
+    # 5) Install/Update database structure (make migrations + migrate)
+    if database_only:
+        num += 1
         printFullTerm(Colors.BLUE, f"{num}) {ACTION} DATABASE")
-        if UPDATE: 
+        if UPDATE:
             _updatedb() or die()
         else:
-           res = install_database(VENV)
-           if(res==0):
-               print(f"You can connect to PyROS as '{SQL_USER}' user with the password '{SQL_PSWD}' ")
+            res = install_database(VENV)
+            if(res == 0):
+                print(
+                    f"You can connect to PyROS as '{SQL_USER}' user with the password '{SQL_PSWD}' ")
     return True
-
-
 @pyros_launcher.command(help="Run some tests")
 @click.option('--app', '-a', help='app name')
 def test(app):
     print("Running tests")
-    os.environ["PATH_TO_OBSCONF_FOLDER"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"obsconfig/fixtures/")
+    os.environ["PATH_TO_OBSCONF_FOLDER"] = os.path.join(
+        os.path.abspath(PYROS_DJANGO_BASE_DIR), "obsconfig/fixtures/")
     os.environ["unit_name"] = ""
     configfile = 'config_pyros.yml'
-    os.environ["pyros_config_file"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../config/pyros/", configfile)
+    os.environ["pyros_config_file"] = os.path.join(os.path.abspath(
+        PYROS_DJANGO_BASE_DIR), "../../../config/pyros/", configfile)
     #start_dir = os.getcwd()
     if app == None:
         #apps = ['obsconfig','scientific_program','common', 'scheduler', 'routine_manager', 'user_manager', 'alert_manager.tests.TestStrategyChange']
         # Removing alert_manager, scheduler from tests
-        apps = ['obsconfig',"scientific_program",'common',  'user_manager', 'routine_manager']
+        apps = ['obsconfig', "scientific_program",
+                'common',  'user_manager', 'routine_manager']
     else:
-        os.environ["PATH_TO_OBSCONF_FILE"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"obsconfig/fixtures/observatory_configuration_ok_simple.yml")
+        os.environ["PATH_TO_OBSCONF_FILE"] = os.path.join(os.path.abspath(
+            PYROS_DJANGO_BASE_DIR), "obsconfig/fixtures/observatory_configuration_ok_simple.yml")
         change_dir(PYROS_DJANGO_BASE_DIR)
-        execProcessFromVenv('manage.py test --keep --noinput --parallel 4 ' + app) or die()
+        execProcessFromVenv(
+            'manage.py test --keep --noinput --parallel 4 ' + app) or die()
         change_dir("PREVIOUS")
         return True
     for app in apps:
         if app != "obsconfig":
             # for every module except obsconfig, should use the simple observatory configuration in order to run the website with a configuration that isn't the observatory configuration used for production
-            os.environ["PATH_TO_OBSCONF_FILE"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"obsconfig/fixtures/observatory_configuration_ok_simple.yml")
+            os.environ["PATH_TO_OBSCONF_FILE"] = os.path.join(os.path.abspath(
+                PYROS_DJANGO_BASE_DIR), "obsconfig/fixtures/observatory_configuration_ok_simple.yml")
         #_loaddata() or die()
         change_dir(PYROS_DJANGO_BASE_DIR)
         # Delete test_pyros database after tests
@@ -608,6 +628,7 @@ def test(app):
     # execProcess("python install.py install")
     return True
+
 @pyros_launcher.command(help="Run ALL tests")
 def testall():
     change_dir(PYROS_DJANGO_BASE_DIR)
@@ -619,8 +640,6 @@ def testall():
     return True
-
-    
 @pyros_launcher.command(help="Update (only if necessary) the python packages AND the source code AND the DB structure")
 def update():
     install_or_update(UPDATE=True)
@@ -653,13 +672,16 @@ def update():
 def _gitpull():
     print("-- running git pull")
     GIT = "git.exe" if IS_WINDOWS else "git"
-    if not test_mode(): return execProcess(f"{GIT} pull")
+    if not test_mode():
+        return execProcess(f"{GIT} pull")
     return True
-#@pyros_launcher.command(help="Update the pyros database")
+# @pyros_launcher.command(help="Update the pyros database")
+
+
 def _updatedb():
     print("-- update db (make migrations + migrate)")
-    if not test_mode() :
+    if not test_mode():
         _makemigrations() or die()
         _migrate() or die()
     return True
@@ -668,7 +690,7 @@ def _updatedb():
 @pyros_launcher.command(help="Update the pyros database and fill it with initial fixture data")
 def initdb():
     if not test_mode():
-        #updatedb()
+        # updatedb()
         res1 = _makemigrations()
         res2 = _migrate()
         res3 = _loaddata()
@@ -677,15 +699,15 @@ def initdb():
 @pyros_launcher.command(help="Launch an agent")
-#@global_test_options
+# @global_test_options
 @click.argument('agent')
 @click.option('--configfile', '-c', help='the configuration file to be used')
 @click.option('--observatory', '-o', help='the observatory name to be used')
 @click.option('--unit', '-u', help='the unit name to be used')
-#@click.option('--format', '-f', type=click.Choice(['html', 'xml', 'text']), default='html', show_default=True)
-#@click.option('--port', default=8000)
-#def start(agent:str, configfile:str, test, verbosity):
-def start(agent:str, configfile:str, observatory:str, unit:str):
+# @click.option('--format', '-f', type=click.Choice(['html', 'xml', 'text']), default='html', show_default=True)
+# @click.option('--port', default=8000)
+# def start(agent:str, configfile:str, test, verbosity):
+def start(agent: str, configfile: str, observatory: str, unit: str):
     log.debug("Running start command")
     try:
         from config.pyros.config_pyros import ConfigPyros
@@ -693,32 +715,41 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
         #pip = "pip" if platform.system() == "Windows" else "pip3"
         # TODO: "python -m pip" au lieu de "pip"
         pip = "pip" if IS_WINDOWS else "pip3"
-        process = subprocess.Popen(pip + " install --upgrade pykwalify", shell=True)
+        process = subprocess.Popen(
+            pip + " install --upgrade pykwalify", shell=True)
         process.wait()
         if process.returncode == 0:
             # self.addExecuted(self.current_command, command)
             from config.pyros.config_pyros import configpyros
         else:
-            log.error("pykwalify package (required for obsconfig class) installation failed")
-    if configfile: 
-        log.debug("With config file"+ configfile)
-        os.environ["pyros_config_file"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../config/pyros/", configfile)
-    else: 
+            log.error(
+                "pykwalify package (required for obsconfig class) installation failed")
+    if configfile:
+        log.debug("With config file" + configfile)
+        os.environ["pyros_config_file"] = os.path.join(os.path.abspath(
+            PYROS_DJANGO_BASE_DIR), "../../../config/pyros/", configfile)
+    else:
         configfile = 'config_pyros.yml'
-        os.environ["pyros_config_file"] = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../config/pyros/", configfile)
-        
-    logo_name = ConfigPyros(os.environ["pyros_config_file"]).pyros_config["general"]["logo"]
-    logo_path = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../config/pyros/", logo_name)
-    media_path = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"misc/static/media",logo_name)
-    shutil.copy(logo_path,media_path)
+        os.environ["pyros_config_file"] = os.path.join(os.path.abspath(
+            PYROS_DJANGO_BASE_DIR), "../../../config/pyros/", configfile)
+
+    logo_name = ConfigPyros(
+        os.environ["pyros_config_file"]).pyros_config["general"]["logo"]
+    logo_path = os.path.join(os.path.abspath(
+        PYROS_DJANGO_BASE_DIR), "../../../config/pyros/", logo_name)
+    media_path = os.path.join(os.path.abspath(
+        PYROS_DJANGO_BASE_DIR), "misc/static/media", logo_name)
+    shutil.copy(logo_path, media_path)
     #if test_mode(): print("in test mode")
     #if verbose_mode(): print("in verbose mode")
     if observatory == None or len(observatory) == 0:
         observatory = "default"
-    observatories_configuration_folder = os.path.join(os.path.abspath(PYROS_DJANGO_BASE_DIR),"../../../privatedev/config/")
+    observatories_configuration_folder = os.path.join(
+        os.path.abspath(PYROS_DJANGO_BASE_DIR), "../../../privatedev/config/")
     if len(glob.glob(observatories_configuration_folder+observatory+"/")) != 1:
         # Observatory configuration folder not found
-        print(f"Observatory configuration folder for observatory '{observatory}' not found in {observatories_configuration_folder}")
+        print(
+            f"Observatory configuration folder for observatory '{observatory}' not found in {observatories_configuration_folder}")
         print("Available observatories configuration :")
         for obs_conf_folder in os.listdir(observatories_configuration_folder):
             print(obs_conf_folder)
@@ -726,10 +757,12 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
     path_to_obs_config_folder = observatories_configuration_folder+observatory+"/"
     obs_config_file_name = ""
-    # Search for observatory config file 
-    obs_config_file_name = glob.glob(path_to_obs_config_folder+"/observatory*.yml")[0]
-    
-    obs_config_file_path = os.path.join(path_to_obs_config_folder,obs_config_file_name)
+    # Search for observatory config file
+    obs_config_file_name = glob.glob(
+        path_to_obs_config_folder+"/observatory*.yml")[0]
+
+    obs_config_file_path = os.path.join(
+        path_to_obs_config_folder, obs_config_file_name)
     os.environ["PATH_TO_OBSCONF_FILE"] = obs_config_file_path
     os.environ["PATH_TO_OBSCONF_FOLDER"] = path_to_obs_config_folder
     os.environ["unit_name"] = unit if unit else ''
@@ -739,17 +772,18 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
     else:
         os.environ["unit_name"] = ""
     '''
-    
+
     # add path to pyros_django folder as the config class is supposed to work within this folder
     #cmd_test_obs_config = f"-c \"from src.core.pyros_django.obsconfig.obsconfig_class import OBSConfig\nOBSConfig('{os.path.join(PYROS_DJANGO_BASE_DIR,os.environ.get('PATH_TO_OBSCONF_FILE'))}')\""
     cmd_test_obs_config = f"-c \"from src.core.pyros_django.obsconfig.obsconfig_class import OBSConfig\nOBSConfig('{obs_config_file_path}')\""
     if not execProcessFromVenv(cmd_test_obs_config):
         # Observatory configuration has an issue
-        exit(1) 
-    # Check each agent in the given list    
+        exit(1)
+    # Check each agent in the given list
     agents = agent.split(",") if "," in agent else [agent]
-    for a in agents: 
-        if not _check_agent(a): return
+    for a in agents:
+        if not _check_agent(a):
+            return
     #print("Agents are:", agents)
     '''
     # Check if multiple agents:
@@ -764,15 +798,15 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
         agents = [agent]
         if not _check_agent(agent): return
     '''
-    
+
     # Start Agents (processes)
     current_processes = []
-    for agent_name,agent_folder in AGENTS.items():
+    for agent_name, agent_folder in AGENTS.items():
-        #if agent in ("all", agent_name) :
-        if agent=="all" or agent_name in agents:
+        # if agent in ("all", agent_name) :
+        if agent == "all" or agent_name in agents:
             # Default case, launch agentX
-            #if agent_name == "agentX":
+            # if agent_name == "agentX":
             # execProcessFromVenvAsync(VENV_PYTHON + " manage.py runserver")
             log.debug(VENV_PYTHON)
@@ -783,22 +817,22 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
             # OLD format agents: majordome, monitoring, alert...
             cmd = "start_agent.py " + agent_name + " " + configfile
-            
+
             # Agent "webserver"
-            if agent_name == "webserver": 
+            if agent_name == "webserver":
                 cmd = "manage.py runserver"
                 if(WITH_DOCKER):
                     # If we're running pyros within docker, we need to specify a specific adress in order to access the website on our host machine
                     PYROS_WEBSITE_PORT = os.environ.get("PYROS_WEBSITE_PORT")
-                    cmd=f"manage.py runserver 0.0.0.0:{PYROS_WEBSITE_PORT}"
+                    cmd = f"manage.py runserver 0.0.0.0:{PYROS_WEBSITE_PORT}"
                 os.chdir(PYROS_DJANGO_BASE_DIR)
             #if not test_mode(): execProcessFromVenv("start_agent.py " + agent_name + " " + configfile)
-            # Agent "agentM", "agentA", "agentB", "agentX", ...            
+            # Agent "agentM", "agentA", "agentB", "agentX", ...
             elif agent_name.startswith("agent"):
                 # Run agent without actual commands sent to devices (FOR_REAL=False)
-                ##agentX.run(FOR_REAL=True)
-                if agent_name == "agentM": 
+                # agentX.run(FOR_REAL=True)
+                if agent_name == "agentM":
                     os.chdir(PYROS_DJANGO_BASE_DIR+"/monitoring/")
                 elif agent_name == "agentSP":
                     os.chdir(PYROS_DJANGO_BASE_DIR+"/scientific_program/")
@@ -807,53 +841,62 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
                 #cmd = "-m AgentX"
                 #cmd = f" Agent{agent_name[5:]}.py {configfile}"
                 cmd = f"Agent{agent_name[5:]}.py"
-                if debug_mode(): cmd += " -d"
-                if sim_mode(): cmd += " -s"
-                if test_mode(): cmd += " -t"
-                if verbose_mode(): cmd += " -v"
-                if configfile: cmd += " {configfile}"
-                
+                if debug_mode():
+                    cmd += " -d"
+                if sim_mode():
+                    cmd += " -s"
+                if test_mode():
+                    cmd += " -t"
+                if verbose_mode():
+                    cmd += " -v"
+                if configfile:
+                    cmd += " {configfile}"
+
             #if not test_mode(): current_processes.append( [execProcessFromVenvAsync(cmd), agent_name, -1] )
             # Append this process ( [process id, agent_name, result=failure] )
             # ("result" will be updated at the end of execution)
-            current_processes.append( [execProcessFromVenvAsync(cmd), agent_name, -1] )
+            current_processes.append(
+                [execProcessFromVenvAsync(cmd), agent_name, -1])
             # self.change_dir("..")
             os.chdir(current_dir)
     # Go back to root folder (/)
     # self.change_dir('..')
-    #os.chdir("..")
+    # os.chdir("..")
     # Wait for end of each process execution
-    #for (p,agent) in current_processes:
+    # for (p,agent) in current_processes:
     for process in current_processes:
-        p,agent,_ = process
-        log.debug(f"************ Waiting for end of execution of agent {agent} ************")
+        p, agent, _ = process
+        log.debug(
+            f"************ Waiting for end of execution of agent {agent} ************")
         p.wait()
         process[2] = p.returncode
         print(f"************ END of execution of agent {agent} ************")
         if p.returncode == 0:
-            printFullTerm(Colors.GREEN, f"Process {agent} executed successfully")
+            printFullTerm(
+                Colors.GREEN, f"Process {agent} executed successfully")
             # self.addExecuted(self.current_command, command)
         else:
             printFullTerm(Colors.WARNING, f"Process {agent} execution failed")
             # self.addError(self.current_command, command)
-    
+
     print()
     print()
     print("Synthesis of the results:")
     for process in current_processes:
-        p,agent,returncode = process
+        p, agent, returncode = process
         if returncode == 0:
-            printFullTerm(Colors.GREEN, f"Process {agent} executed successfully")
+            printFullTerm(
+                Colors.GREEN, f"Process {agent} executed successfully")
             # self.addExecuted(self.current_command, command)
         else:
             printFullTerm(Colors.WARNING, f"Process {agent} execution failed")
             # self.addError(self.current_command, command)
-        
+
     #print("************ end of START() ************")
     # Only according to the last process status:
-    #return True if p.returncode==0 else False
-    return True if p.returncode==0 else False
+    # return True if p.returncode==0 else False
+    return True if p.returncode == 0 else False
 # TODO: implémenter le STOP !!!
@@ -861,8 +904,8 @@ def start(agent:str, configfile:str, observatory:str, unit:str):
 @click.argument('agent')
 def stop(agent):
     print("Running stop command")
-    if not _check_agent(agent): return
-
+    if not _check_agent(agent):
+        return
 """
@@ -871,14 +914,17 @@ def stop(agent):
 ********************************************************************************
 """
+
 def notused_update_python_packages_from_requirements():
     # 1) Upgrade pip (if new version available)
     res = execProcessFromVenv("-m pip install --upgrade pip")
-    if not res: return False
+    if not res:
+        return False
     # 2) Install only "not yet installed" python packages
     res = execProcessFromVenv("-m pip install -r install/"+REQUIREMENTS)
     return res
+
 def _update_plantuml_diags():
     for dirpath, dirnames, files in os.walk(PYROS_DJANGO_BASE_DIR):
         if os.path.basename(dirpath) == "doc":
@@ -887,13 +933,34 @@ def _update_plantuml_diags():
             for diag in diagrams:
                 # if no diag.png or if diag.pu more recent than diag.png => re-generate diag.png
                 diag_png = diag.split('.pu')[0] + '.png'
-                if not os.path.isfile(diag_png) or ( os.path.getmtime(diag) > os.path.getmtime(diag_png) ) :
+                if not os.path.isfile(diag_png) or (os.path.getmtime(diag) > os.path.getmtime(diag_png)):
                     #res = execProcessFromVenv("-m plantuml "+diag)
                     # we need to use our own version of plantuml
                     res = execProcessFromVenv("./install/plantuml.py "+diag)
-                    if not res: return False
+                    if not res:
+                        return False
     return True
+
+def _update_api_doc():
+    print(os.getcwd())
+    DOC_RST_PATH = "doc/doc_rst/"
+    # 1) install/update Sphinx requirements (only if not yet installed)
+    venv_pip_install2(DOC_RST_PATH+'requirements.txt', '-r')
+    # 2) make html doc from RST doc
+    # cd doc/sourcedoc/
+    change_dir(DOC_RST_PATH)
+    # ./make_rst_then_html
+    res = execProcess('make clean ; /bin/bash make_rst_then_html')
+    print("The technical (and API) documentation has been generated and can be opened in a local browser with this command :")
+    print("     open "+DOC_RST_PATH+"build/html/index.html")
+    print("     (or just open your local browser _firefox, chrome, ..._ and go to this file)")
+    # Come back to where we were before
+    # cd -
+    change_dir("PREVIOUS")
+    return True
+
+
 def _migrate():
     change_dir(PYROS_DJANGO_BASE_DIR)
     # Migrate only migrations for the app "common"
@@ -903,6 +970,7 @@ def _migrate():
     change_dir("PREVIOUS")
     return res
+
 def _makemigrations():
     change_dir(PYROS_DJANGO_BASE_DIR)
     #execProcessFromVenv(self.venv_bin + " manage.py makemigrations")
@@ -911,52 +979,54 @@ def _makemigrations():
     change_dir("PREVIOUS")
     return res
-#TODO: mettre la fixture en date naive (sans time zone)
+# TODO: mettre la fixture en date naive (sans time zone)
+
+
 def _loaddata():
     change_dir(PYROS_DJANGO_BASE_DIR)
     #execProcessFromVenv(self.venv_bin + " manage.py loaddata misc" + os.sep + "fixtures" + os.sep + self.INIT_FIXTURE)
-    res = execProcessFromVenv("manage.py loaddata misc" + os.sep + "fixtures" + os.sep + INIT_FIXTURE)
+    res = execProcessFromVenv(
+        "manage.py loaddata misc" + os.sep + "fixtures" + os.sep + INIT_FIXTURE)
     change_dir("PREVIOUS")
     return res
 def change_dir(path):
     global _previous_dir
-    if path == "PREVIOUS": path=_previous_dir
+    if path == "PREVIOUS":
+        path = _previous_dir
     _previous_dir = os.getcwd()
     log.debug("Moving to : " + path)
     os.chdir(path)
     log.debug("Current directory : " + str(os.getcwd()))
-
 def _check_agent(agent):
     # Check that agent exists
-    if agent != "all" and agent not in AGENTS.keys() :
+    if agent != "all" and agent not in AGENTS.keys():
         # Check if multiple agents:
         print("This agent does not exist")
         print("Here is the allowed list of agents:")
         print("- all => will launch ALL agents")
-        for agent_name in AGENTS.keys(): print('-',agent_name)
+        for agent_name in AGENTS.keys():
+            print('-', agent_name)
         return False
     return True
-
-
-
-
 """
 INSTALLATION FUNCTIONS (./pyros.py install)
 """
+
 def replacePatternInFile(pattern, replace, file_path):
     try:
         with fileinput.FileInput(file_path, inplace=True, backup='.bak') as file:
             for line in file:
                 print(line.replace(pattern, replace), end='')
     except:
-        sys.stderr.write(Colors.ERROR + "ERROR !: replacement in file failed !" + Colors.END + "\r\n")
+        sys.stderr.write(
+            Colors.ERROR + "ERROR !: replacement in file failed !" + Colors.END + "\r\n")
         return 1
     return 0
@@ -975,7 +1045,8 @@ def notused_install_dependency_ubuntu(command, mode):
     process = subprocess.Popen(command, shell=True)
     process.wait()
     if process.returncode != 0:
-        sys.stderr.write(Colors.ERROR + "ERROR !: installation of " + old + " failed !" + Colors.END + "\r\n")
+        sys.stderr.write(Colors.ERROR + "ERROR !: installation of " +
+                         old + " failed !" + Colors.END + "\r\n")
 def notused_install_required_ubuntu():
@@ -998,7 +1069,8 @@ def notused_install_dependency_centos(command, mode):
     process = subprocess.Popen(command, shell=True)
     process.wait()
     if process.returncode != 0:
-        sys.stderr.write(Colors.ERROR + "ERROR !: installation of " + old + " failed !" + Colors.END + "\r\n")
+        sys.stderr.write(Colors.ERROR + "ERROR !: installation of " +
+                         old + " failed !" + Colors.END + "\r\n")
 def notused_install_required_centos():
@@ -1034,7 +1106,8 @@ def notused_install_required():
     # Find the linux distribution and call the related function
     distribution = platform.dist()
     if not 'SUDO_UID' in os.environ.keys():
-        sys.stderr.write("Super user rights are needed to install prerequisites\r\n")
+        sys.stderr.write(
+            "Super user rights are needed to install prerequisites\r\n")
         exit(1)
     if distribution[0] == "Ubuntu" or distribution[0] == "Debian":
         install_required_ubuntu()
@@ -1045,11 +1118,16 @@ def notused_install_required():
         exit(1)
-def venv_pip_install(package_name:str, options:str=''):
+def venv_pip_install(package_name: str, options: str = ''):
     os.system(VENV_PIP + ' install ' + options + ' ' + package_name)
-def install_venv(EVEN_IF_ALREADY_EXISTS:bool=False):
+def venv_pip_install2(package_name: str, options: str = ''):
+    os.system(VENV_PYTHON + ' -m ' + VENV_PIP + ' install '
+              + options + ' ' + package_name)
+
+
+def install_venv(EVEN_IF_ALREADY_EXISTS: bool = False):
     '''
     Install VENV if does not exist 
     OR 
@@ -1072,14 +1150,14 @@ def install_venv(EVEN_IF_ALREADY_EXISTS:bool=False):
     # --- (If not exists) Create the (private) venv ROOT directory (where the virtual environment dir will be created)
     # --------------------------------------------
     if (os.path.basename(os.getcwd()) != VENV_ROOT):
-        ##if not(os.path.isdir("../venv")):
+        # if not(os.path.isdir("../venv")):
         if not(os.path.isdir(VENV_ROOT)):
-            print(Colors.LOG_BLUE + f"-----------------------------Creating 'venv' root directory ({VENV_ROOT}/)-----------------------------" + Colors.END)
-            #os.mkdir("../venv")
+            print(Colors.LOG_BLUE +
+                  f"-----------------------------Creating 'venv' root directory ({VENV_ROOT}/)-----------------------------" + Colors.END)
+            # os.mkdir("../venv")
             os.mkdir(VENV_ROOT)
     os.chdir(VENV_ROOT)
-    
     # --------------------------------------------
     # --- (if EVEN_IF_ALREADY_EXISTS) Delete venv dir if already exist
     # --------------------------------------------
@@ -1088,20 +1166,22 @@ def install_venv(EVEN_IF_ALREADY_EXISTS:bool=False):
         while True:
             try:
                 if os.path.isdir(VENV):
-                    print(Colors.LOG_BLUE + f"-----------------------------Deleting existing VENV dir ({VENV}/)-----------------------------" + Colors.END)
+                    print(
+                        Colors.LOG_BLUE + f"-----------------------------Deleting existing VENV dir ({VENV}/)-----------------------------" + Colors.END)
                     shutil.rmtree(VENV)
                 break
             # Exception on Windows WinError 145 : Cannot remove folder because files in folder not yet removed...
             except Exception as e:
-                #print(e)
+                # print(e)
                 continue
     # --------------------------------------------
     # --- (if not exists) Reinstall the virtual environment (from ../venv/)
     # --------------------------------------------
-    
+
     if not os.path.isdir(VENV):
-        print(Colors.LOG_BLUE + f"-----------------------------Creating venv dir ({VENV}/)-----------------------------"+END_OF_LINE + Colors.END)
+        print(Colors.LOG_BLUE +
+              f"-----------------------------Creating venv dir ({VENV}/)-----------------------------"+END_OF_LINE + Colors.END)
         os.system(GLOBAL_PYTHON+" -m venv " + VENV)
     # Come back to project root
@@ -1113,7 +1193,8 @@ def install_packages():
     os.chdir(VENV_ROOT)
-    print(Colors.LOG_BLUE + "-----------------------------Upgrade pip, wheel, and setuptools" + "-----------------------------"+END_OF_LINE + Colors.END)
+    print(Colors.LOG_BLUE + "-----------------------------Upgrade pip, wheel, and setuptools" +
+          "-----------------------------"+END_OF_LINE + Colors.END)
     # Upgrade pip
     os.system(VENV_PYTHON + ' -m pip install --upgrade pip')
     '''
@@ -1122,14 +1203,14 @@ def install_packages():
     else: # Linux
         os.system(venv + '/bin/python -m pip install --upgrade pip')
     '''
-    
+
     # Pip upgrade wheel and setuptools
     venv_pip_install('wheel', '--upgrade')
     #os.system(VENV_PIP+' install --upgrade wheel')
     # Not working with python 3.8 (on 17/02/2022)
     # venv_pip_install('setuptools', '--upgrade')
     os.system(VENV_PIP+' install setuptools==58')
-         
+
     # Pip install required packages from REQUIREMENTS file
     print()
     print(Colors.LOG_BLUE + "-----------------------------Installing python packages via pip-----------------------------" + Colors.END)
@@ -1140,11 +1221,12 @@ def install_packages():
     if IS_WINDOWS:
         os.chdir("../install")
-        ## moving voeventparse in site-packages directory
+        # moving voeventparse in site-packages directory
         try:
             site_packages = "..\\venv\\"+VENV+"\\Lib\\site-packages\\"
             if (
-                not os.path.isdir(site_packages + "voevent_parse-0.9.5.dist-info")
+                not os.path.isdir(
+                    site_packages + "voevent_parse-0.9.5.dist-info")
                 and
                 not os.path.isdir(site_packages + "voeventparse")
             ):
@@ -1152,19 +1234,24 @@ def install_packages():
                 cmdline = "xcopy /i /y windows\\voeventparse " + site_packages + "voeventparse"
                 process = subprocess.Popen(cmdline)
                 process.wait()
-                if (process.returncode != 0): raise Exception
-                process = subprocess.Popen("xcopy /i /y windows\\voevent_parse-0.9.5.dist-info " + site_packages + "voevent_parse-0.9.5.dist-info")
+                if (process.returncode != 0):
+                    raise Exception
+                process = subprocess.Popen(
+                    "xcopy /i /y windows\\voevent_parse-0.9.5.dist-info " + site_packages + "voevent_parse-0.9.5.dist-info")
                 process.wait()
-                if (process.returncode != 0): raise Exception
+                if (process.returncode != 0):
+                    raise Exception
                 print(Colors.LOG_BLUE + "\r\n-----------------------------library successfully copied-----------------------------" + Colors.END)
         except Exception as e:
-            print(Colors.ERROR + "ERROR while Copying the voevent library in Lib/site-packages" + Colors.END) ; #, file=stderr)
+            # , file=stderr)
+            print(
+                Colors.ERROR + "ERROR while Copying the voevent library in Lib/site-packages" + Colors.END)
             return False
-    
+
     # Go back to project root dir
     os.chdir('..')
-    
-    #return 0
+
+    # return 0
     return True
@@ -1172,9 +1259,10 @@ def install_database(venv):
     SQL_USER = os.environ.get("MYSQL_PYROS_LOGIN").strip()
     SQL_PSWD = os.environ.get("MYSQL_PYROS_PWD").strip()
-    
-    print(Colors.LOG_BLUE + END_OF_LINE+"-----------------------------Launching mysql to create database and create and grant user pyros-----------------------------" + Colors.END)
-  
+
+    print(Colors.LOG_BLUE + END_OF_LINE +
+          "-----------------------------Launching mysql to create database and create and grant user pyros-----------------------------" + Colors.END)
+
     # --------------------------------------------
     # --- Determine the MySQL version
     # --------------------------------------------
@@ -1211,9 +1299,9 @@ def install_database(venv):
         f"DROP DATABASE {IF_EXISTS} {SQL_DATABASE_SIMU}; CREATE DATABASE {SQL_DATABASE_SIMU}; " +\
         f"DROP DATABASE {IF_EXISTS} {SQL_DATABASE_TEST}; CREATE DATABASE {SQL_DATABASE_TEST}; "
-    # 2) Create user pyros and give it all rights on 3 databases 
+    # 2) Create user pyros and give it all rights on 3 databases
     # Ne marche pas si l'utilisateur existe déjà => erreur
-    #"CREATE USER "+SQL_USER+"; " +\
+    # "CREATE USER "+SQL_USER+"; " +\
     # Donc, il faut ruser
     # Mysql >= 5.7 only:
     #sql_query_elem = "CREATE USER IF NOT EXISTS "+SQL_USER+"; "
@@ -1227,7 +1315,7 @@ def install_database(venv):
     else:
         host = "localhost"
     # FOR MYSQL 8:
-    if sql_version >=8:
+    if sql_version >= 8:
         # We remove identified by because grant was creating the pyros user, wasn't working on version 8 so we separated the steps :
         # first we create the user then we grant him rights
         sql_query += f"CREATE USER IF NOT EXISTS '{SQL_USER}'@'{host}' IDENTIFIED BY '{SQL_PSWD}'; "
@@ -1237,7 +1325,7 @@ def install_database(venv):
     else:
         sql_query += f"GRANT ALL ON {SQL_DATABASE}.* TO '{SQL_USER}'@'{host}' IDENTIFIED BY '{SQL_PSWD}'; "
         sql_query += f"GRANT ALL ON {SQL_DATABASE_SIMU}.* TO '{SQL_USER}'@'{host}' IDENTIFIED BY '{SQL_PSWD}'; "
-        # This database does not yet exists and will be automatically created by Django, but we already give access rights to it for pyros user 
+        # This database does not yet exists and will be automatically created by Django, but we already give access rights to it for pyros user
         sql_query += f"GRANT ALL ON {SQL_DATABASE_TEST}.* TO '{SQL_USER}'@'{host}' IDENTIFIED BY '{SQL_PSWD}'; "
     # FOR MYSQL 8:
     if sql_version >= 8:
@@ -1259,12 +1347,12 @@ def install_database(venv):
     #req = "drop database pyros ; CREATE DATABASE pyros; drop database pyros_test ; CREATE DATABASE pyros_test; DROP USER 'pyros'@'localhost' ; GRANT USAGE ON *.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION; DROP USER 'pyros'@'localhost'; GRANT ALL ON pyros.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros'; GRANT ALL PRIVILEGES ON pyros.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION;  GRANT ALL PRIVILEGES ON pyros_test.* TO pyros@localhost IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION;"
     # (EP) ok for CENTOS 7 I suppose (but not for CentOS 6):
     #req_centos = "CREATE DATABASE IF NOT EXISTS pyros; CREATE DATABASE IF NOT EXISTS pyros_test; GRANT USAGE ON *.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION; DROP USER 'pyros'@'localhost'; GRANT ALL ON pyros.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros'; GRANT ALL PRIVILEGES ON pyros.* TO 'pyros'@'localhost' IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION;  GRANT ALL PRIVILEGES ON pyros_test.* TO pyros@localhost IDENTIFIED BY 'DjangoPyros' WITH GRANT OPTION;"
-    
+
     # --- Prepare the SQL query to create and initialize the pyros database if needed
-    #if platform.dist()[0] == "centos":
+    # if platform.dist()[0] == "centos":
     #    req = sql_query
     #mysql_call_root = '"' + MYSQL_EXE_PATH + 'mysql" -u root -p'
-    
+
     # if the mysql_root_password isn't defined, it will ask to enter the password
     try:
         # a root password should be defined as an environment variable
@@ -1284,48 +1372,52 @@ def install_database(venv):
     # when using -p option, there is no space between the option and the password value
     mysql_call_root = f"mysql -h {mysql_host} -u '{mysql_login}' -p'{mysql_root_password}'"
-    mysql_call_pyros = "\"" + MYSQL_EXE_PATH+ "mysql\" -u "+SQL_USER+" -p"
-    
+    mysql_call_pyros = "\"" + MYSQL_EXE_PATH + "mysql\" -u "+SQL_USER+" -p"
+
     # --------------------------------------------
     # --- Creating database and creating and granting user pyros
     # --------------------------------------------
     user_ros_is_created = True
-    if sql_version<5.5:
-       print(Colors.LOG_BLUE +"------------------ Check if the user pyros exists in MYSQL (type the pyros password) -----------------------------" + Colors.END)
-       # --- We are testing if user pyros already exists in the database
-       process = subprocess.Popen("echo quit |" + mysql_call_pyros, shell=True)
-       process.wait()
-       if (process.returncode == 0):
-           user_ros_is_created = False
+    if sql_version < 5.5:
+        print(Colors.LOG_BLUE + "------------------ Check if the user pyros exists in MYSQL (type the pyros password) -----------------------------" + Colors.END)
+        # --- We are testing if user pyros already exists in the database
+        process = subprocess.Popen(
+            "echo quit |" + mysql_call_pyros, shell=True)
+        process.wait()
+        if (process.returncode == 0):
+            user_ros_is_created = False
     if user_ros_is_created:
-       # --- The user pyros must be created in the database
-       print(Colors.LOG_BLUE +"-----------------------------Please enter your MYSQL root password-----------------------------" + Colors.END)
-       #process = subprocess.Popen("echo \"" + sql_query + "\" |"+ mysql_call_root, shell=True)
-       sql_cmd = 'echo "' + sql_query + '" | '+ mysql_call_root
-       print("Executing sql cmd: ", sql_cmd)
-       process = subprocess.Popen(sql_cmd, shell=True)
-       process.wait()
+        # --- The user pyros must be created in the database
+        print(Colors.LOG_BLUE + "-----------------------------Please enter your MYSQL root password-----------------------------" + Colors.END)
+        #process = subprocess.Popen("echo \"" + sql_query + "\" |"+ mysql_call_root, shell=True)
+        sql_cmd = 'echo "' + sql_query + '" | ' + mysql_call_root
+        print("Executing sql cmd: ", sql_cmd)
+        process = subprocess.Popen(sql_cmd, shell=True)
+        process.wait()
     if (process.returncode != 0):
-        sys.stderr.write(Colors.ERROR + "ERROR !: db configuration failed !" + Colors.END + "\r\n")
+        sys.stderr.write(
+            Colors.ERROR + "ERROR !: db configuration failed !" + Colors.END + "\r\n")
         return -1
-    print(Colors.LOG_BLUE + END_OF_LINE+"-----------------------------Database created and user pyros successfully created and granted-----------------------------" + Colors.END)
+    print(Colors.LOG_BLUE + END_OF_LINE +
+          "-----------------------------Database created and user pyros successfully created and granted-----------------------------" + Colors.END)
     # --------------------------------------------
     # --- Replacing pattern in settings.py to use mysql
     # --------------------------------------------
     print(Colors.LOG_BLUE + "-----------------------------setting MYSQL = True in settings-----------------------------" + Colors.END)
-    replacePatternInFile("MYSQL = False", "MYSQL = True", os.path.normpath("src/core/pyros_django/pyros/settings.py"))
-    
+    replacePatternInFile("MYSQL = False", "MYSQL = True", os.path.normpath(
+        "src/core/pyros_django/pyros/settings.py"))
+
     #print(Colors.LOG_BLUE + "\r\n-----------------------------cd ..-----------------------------" + Colors.END)
     # 2021 : no need to change folder in order to init the database (maybe the next line was forget to be commented because the previous line is commented)
-    #os.chdir("..")
+    # os.chdir("..")
     # --------------------------------------------
     # ---  Executing migrations
     # --------------------------------------------
-    
+
     print(Colors.LOG_BLUE + "\r\n\r\n-----------------------------Migrate : executing pyros.py init_database-----------------------------" + Colors.END)
-    #TODO: from venv !!!
+    # TODO: from venv !!!
     try:
         #os.system(GLOBAL_PYTHON+" pyros.py init_database")
         res = os.system(GLOBAL_PYTHON+" pyros.py initdb")
@@ -1345,46 +1437,37 @@ def install_database(venv):
 def notused_help():
     print(
-        #"Welcome in the installation script of the pyros venv.\t\nPlease launch it from the install directory of pyros.\n\tIf you're on Ubuntu Debian or CentOS:\n\tlaunch it with sudo and <--prerequisites> or <-p> to install the prerequisites.\n\t-->sudo ./test_install.py -p\n\n\tFor the python packages launch it from the install directory of pyros without sudo and without parameter\n\t-->./test_install.py")
+        # "Welcome in the installation script of the pyros venv.\t\nPlease launch it from the install directory of pyros.\n\tIf you're on Ubuntu Debian or CentOS:\n\tlaunch it with sudo and <--prerequisites> or <-p> to install the prerequisites.\n\t-->sudo ./test_install.py -p\n\n\tFor the python packages launch it from the install directory of pyros without sudo and without parameter\n\t-->./test_install.py")
         "Welcome to the installation script of the pyros software.\t\n" + \
         "Usage:\n" + \
         "  [python] ./pyros.py install [-p] [-d]"
     )
-
-
-
-
-
-
-
-
-
-
 """
 ********************************************************************************
 ********************************* main() FUNCTION ******************************
 ********************************************************************************
 """
+
+
 def main():
-    set_environment_variables_if_not_configured(ENV_PATH,ENV_SAMPLE_PATH)
+    set_environment_variables_if_not_configured(ENV_PATH, ENV_SAMPLE_PATH)
     pyros_launcher()
-    
-    
+
 # AVIRER
-#@click.command()
-#@click.argument('start')
+# @click.command()
+# @click.argument('start')
 def oldmain():
     '''
     cli()
     return
     '''
-    
-    #fire.Fire(Commands)
-    #return
+
+    # fire.Fire(Commands)
+    # return
     # if len(sys.argv) == 3 and sys.argv[1].startswith("simulator"): SIMULATOR_CONFIG_FILE = sys.argv[2]
     # print(sys.argv)
@@ -1424,8 +1507,8 @@ def oldmain():
     # sys.exit(pyros.exec())
-
-if __name__ == "__main__": main()
+if __name__ == "__main__":
+    main()
 """ INSTALLATION main function