README.md

@@ -29,7 +29,7 @@ Setup & debug
 + ``pip list`` (pip show abibas, pip uninstall abibas, etc)
 
 At this stage the package is installed in editable mode, to run it in debug mode:
-,
+
 + ``abibas-debug``
 
 or in normal mode:
@@ -40,9 +40,3 @@ If in debug mode changes made to the Flask app will be applied automatically at
 for the changes in simulator though the tool has to be restarted.
 
 **main** module can be manipulated for debugging purposes as well
-
-Before releasing
-================
-+ ``pip install twine``
-+ ``twine check dist/*``
-+ ``twine upload --repository-url https://test.pypi.org/legacy/ dist/*``

pyproject.toml

@@ -25,7 +25,6 @@ dependencies = [
     "Flask",
     "pyModbusTCP",
     "prettytable",
-    "plotext",
 ]
 version = "0.0.1"
 urls = {homepage = "https://gittar.crabdance.com/e-corp/abibas"}

src/data/__init__.py

@@ -34,9 +34,6 @@ def _get_controller_models_fname() -> str:
 def _get_reg_value(controller: dict, reg: dict) -> int:
     # get first value from type definition (as default)
     value = None
-    if 'PossibleValues' not in reg or not reg['PossibleValues']:
-        value = 0
-    else:
     k, v = next(iter(reg['PossibleValues'][0].items()), None)
     value = v
     # if there is a file with overriden value, take that
@@ -54,8 +51,8 @@ def _get_reg_value(controller: dict, reg: dict) -> int:
         entry = [x for x in overriden if x['Name'] == reg['Name']]
         if entry:
             value = entry[0]['Value']
-            logging.debug(
-                f'{reg["Name"]} value gotten: {value} from {overrides}')
+            # logging.debug(
+            #     f'{reg["Name"]} value gotten: {value} from {overrides}')
     return value
 
 
@@ -64,6 +61,11 @@ def _save_overrides(fname: str, overrides: list):
         json.dump(overrides, file, indent=2)
 
 
+def get_version() -> str:
+    version = pkg_resources.get_distribution('abibas').version
+    return version
+
+
 def get_controller(port: int) -> dict:
     controllers = get_controllers()
     models = get_controller_types()
@@ -162,10 +164,3 @@ def remove_controller(port: int) -> bool:
         logging.debug(f'Overrides file {overrides} not found, skip removing')
 
     return True
-
-
-def get_info() -> dict:
-    distribution = pkg_resources.get_distribution('abibas')
-    return dict(version=distribution.version,
-                author='Taryel Hlontsi',
-                license='copyleft (GPLv3)')

src/data/controller-models.json

@@ -1,141 +1,4 @@
 [
-  {
-    "Name": "Ads",
-    "RunAs": "Client",
-    "Registers": [
-      {
-        "CanOverride": false,
-        "Name": "Status2",
-        "Type": "Input",
-        "Address": 0,
-        "DisplayAs": "DropdownVariants",
-        "PossibleValues": [
-          {
-            "NotOperting": 0
-          },
-          {
-            "Operating": 1
-          }
-        ]
-      },
-      {
-        "CanOverride": false,
-        "Name": "Sim160",
-        "Type": "Input",
-        "Address": 928,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "ConfigUnixTime4",
-        "Type": "Input",
-        "Address": 259,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "ConfigUnixTime3",
-        "Type": "Input",
-        "Address": 258,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "ConfigUnixTime2",
-        "Type": "Input",
-        "Address": 257,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "ConfigUnixTime1",
-        "Type": "Input",
-        "Address": 256,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "TurbinesConfigured",
-        "Type": "Input",
-        "Address": 261,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "InterfaceVersion",
-        "Type": "Input",
-        "Address": 260,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "Sim2",
-        "Type": "Input",
-        "Address": 770,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "Sim1",
-        "Type": "Input",
-        "Address": 769,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": false,
-        "Name": "SystemStatusCode",
-        "Type": "Input",
-        "Address": 1,
-        "DisplayAs": "RawBinary",
-        "PossibleValues": []
-      },
-      {
-        "CanOverride": true,
-        "Name": "Interface",
-        "Type": "Holding",
-        "Address": 512,
-        "PossibleValues": [
-          {
-            "Deactivated": 0
-          },
-          {
-            "AircraftDetected": 1
-          },
-          {
-            "AircraftNotDetected": 3
-          }
-        ]
-      },
-      {
-        "CanOverride": false,
-        "Name": "Status",
-        "Type": "Holding",
-        "Address": 0,
-        "PossibleValues": [
-          {
-            "NotOperting": 0
-          },
-          {
-            "Operating": 1
-          }
-        ]
-      }
-    ],
-    "DisplayVariants": [
-      "DropdownVariants",
-      "RawBinary",
-      "RawInteger"
-    ]
-  },
   {
     "Name": "CIP300",
     "RunAs": "Server",
@@ -245,6 +108,43 @@
       }
     ]
   },
+  {
+    "Name": "Ads",
+    "RunAs": "Client",
+    "Registers": [
+      {
+        "CanOverride": true,
+        "Name": "Interface",
+        "Type": "Holding",
+        "Address": 512,
+        "PossibleValues": [
+          {
+            "Deactivated": 0
+          },
+          {
+            "AircraftDetected": 1
+          },
+          {
+            "AircraftNotDetected": 3
+          }
+        ]
+      },
+      {
+        "CanOverride": false,
+        "Name": "Status",
+        "Type": "Holding",
+        "Address": 0,
+        "PossibleValues": [
+          {
+            "NotOperting": 0
+          },
+          {
+            "Operating": 1
+          }
+        ]
+      }
+    ]
+  },
   {
     "Name": "CIP400",
     "RunAs": "Server",

src/data/controllers.json

@@ -11,6 +11,12 @@
     "Ip": "127.0.0.1",
     "Port": 11507
   },
+  {
+    "Type": "Quantec",
+    "Enabled": true,
+    "Ip": "127.0.0.1",
+    "Port": 11701
+  },
   {
     "Type": "Quantec",
     "Enabled": true,
@@ -22,11 +28,5 @@
     "Enabled": true,
     "Ip": "127.0.0.1",
     "Port": 11508
-  },
-  {
-    "Type": "CIP400",
-    "Enabled": true,
-    "Ip": "127.0.0.1",
-    "Port": 11506
   }
 ]

src/main/__init__.py

@@ -11,7 +11,8 @@ def run():
 def run_debug():
     ui = _run_flask_debug()
     print(f'UI started: PID {ui.pid}')
-    _run_simulator('debug')
+    #ui.wait()
+    _run_simulator('info')
 
 
 def _run_flask_debug() -> subprocess.Popen:

src/modbus/simulator.py

@@ -11,7 +11,6 @@ import signal
 import data
 import sys
 import traceback
-import plotext as plt
 
 _RUN_FOR_SEC = 10
 _LOG_LEVEL = 20  # INFO
@@ -19,8 +18,6 @@ _LOG_LEVEL = 20  # INFO
 _statistic = {}
 _start_time = None
 _active = {}
-_inbound_read_freq = {}
-_inbound_write_freq = {}
 
 
 def get_run_options():
@@ -82,38 +79,22 @@ def handle_client(instance, ct: dict):
         filter(lambda x: x['Type'] == 'Holding' and x['CanOverride'] is True,
                registers), None)
 
-    inputRegs = filter(lambda x: x['Type'] == 'Input', registers)
-
     regs = {}
     for r in registers:
         regs[r['Address']] = r['Name']
     info = {'Model': ct['Name'], 'Port': instance.port, 'Registers': regs}
-
     if holdingRead:
         result = instance.read_holding_registers(holdingRead['Address'], 1)
-        if result is None:
+        if not result:
             logging.debug(f'{ct["Name"]} client@{instance.port} no connection')
         else:
             gather('read', 'holding', holdingRead['Address'], instance.port,
                    result[0], info, 'outbound')
-            data.set_controller_register(info['Model'], info['Port'],
-                                         holdingRead['Name'], result[0])
             if holdingWrite:
                 v = holdingWrite['CurrentValue']
                 instance.write_single_register(holdingWrite['Address'], v)
                 gather('write', 'holding', holdingWrite['Address'],
                        instance.port, v, info, 'outbound')
-            for r in list(inputRegs):
-                result = instance.read_input_registers(r['Address'])
-                if result is not None:
-                    gather('read', 'input', r['Address'], instance.port,
-                           result[0], info, 'outbound')
-                    data.set_controller_register(info['Model'], info['Port'],
-                                                 r['Name'], result[0])
-                else:
-                    logging.warning(
-                        f'Input register {r["Name"]} on server {instance.port} is misconfigured. Read result is None'
-                    )
     else:
         logging.warning(
             f'{ct["Name"]} client@{instance.port} is not fully configured')
@@ -205,13 +186,11 @@ def gather(op, registry, address, port, value, info, tag='inbound'):
 
     Returns
     -------
-    None
+    void
     """
 
     global _active
     global _statistic
-    global _inbound_read_freq
-    global _inbound_write_freq
     controller = f'{info["Model"]}@{info["Port"]}'
     try:
         # client port here is a port of incoming connection
@@ -237,104 +216,6 @@ def gather(op, registry, address, port, value, info, tag='inbound'):
     else:
         logging.debug(msg)
 
-    freq_key = f'{op};{registry} {address};{controller}'
-    if tag == 'inbound' and (op == 'read' or op == 'write'):
-        collection = _inbound_read_freq if op == 'read' else _inbound_write_freq
-        entry = collection.get(freq_key)
-        if entry:
-            entry[0] += 1
-        else:
-            collection[freq_key] = {0: 0}
-
-
-def calculate_freq(interval_sec: int, dt: datetime.datetime):
-    global _inbound_read_freq
-    global _inbound_write_freq
-    complete_freq_period(_inbound_read_freq, interval_sec, dt)
-    complete_freq_period(_inbound_write_freq, interval_sec, dt)
-
-
-def complete_freq_period(collection: dict, interval_sec: int,
-                         dt: datetime.datetime):
-
-    format = '%Y-%d-%m %H:%M:%S'
-    dt_str = dt.strftime(format)
-    hour_ago_str = (dt - datetime.timedelta(hours=1)).strftime(format)
-    for k, v in collection.items():
-        v[dt_str] = v[0] / interval_sec
-        v[0] = 0
-        keys_to_del = []
-        # starting from 3.7 insertion order in dict is preserved
-        for entry, _ in v.items():
-            if entry == 0:
-                continue
-            if entry < hour_ago_str:
-                keys_to_del.append(entry)
-            else:
-                break
-        for entry in keys_to_del:
-            del v[entry]
-
-
-def merge_dict(base: dict, incoming: dict) -> dict:
-    result = {**base, **incoming}
-    for key in result.keys():
-        if key in base and key in incoming:
-            if isinstance(base[key], list) and isinstance(incoming[key], list):
-                result[key] = base[key] + incoming[key]
-            elif isinstance(base[key], list):
-                result[key] = [*base[key], incoming[key]]
-            elif isinstance(incoming[key], list):
-                result[key] = [base[key], *incoming[key]]
-            else:
-                result[key] = [base[key], incoming[key]]
-        else:
-            if not isinstance(result[key], list):
-                result[key] = [result[key]]
-    return result
-
-
-def aggregate_freq(collection: dict):
-    for k, v in collection.items():
-        del v[0]
-        # print(f'{k}:')
-        # for timestamp, freq in v.items():
-        #     print(f'{timestamp} {freq} ')
-    merged = {}
-    for k, v in collection.items():
-        merged = merge_dict(merged, v)
-    aggregated = {}
-    for k, v in merged.items():
-        v = round(sum(v) / len(v), 2)
-        aggregated[k] = v
-    return aggregated
-
-
-def report_freq():
-    global _inbound_read_freq
-    global _inbound_write_freq
-
-    reads = aggregate_freq(_inbound_read_freq)
-    writes = aggregate_freq(_inbound_write_freq)
-    if not reads and not writes:
-        return
-
-    plt.date_form('Y-d-m H:M:S', 'M:S')
-    if reads:
-        plt.plot(reads.keys(),
-                 reads.values(),
-                 label='inbound read frequency',
-                 color='blue')
-    if writes:
-        plt.plot(writes.keys(),
-                 writes.values(),
-                 label='inbound write frequency',
-                 color='magenta')
-    plt.title('Inbound Read/Write Frequency')
-    plt.xlabel('Time (max 1 hour long)')
-    plt.ylabel('Operations per second')
-    plt.show()
-
 
 def report():
     global _statistic
@@ -419,12 +300,12 @@ class MaBank(DataBank):
 
     def on_coils_change(self, address, from_value, to_value, srv_info):
         gather('change event', 'coils', address, srv_info.client.port,
-               f'{from_value}→{to_value}', self.info)
+               f'{from_value}⟶{to_value}', self.info)
 
     def on_holding_registers_change(self, address, from_value, to_value,
                                     srv_info):
         gather('change event', 'holding', address, srv_info.client.port,
-               f'{from_value}→{to_value}', self.info)
+               f'{from_value}⟶{to_value}', self.info)
 
 
 if __name__ == '__main__':
@@ -435,7 +316,7 @@ if __name__ == '__main__':
                             level=_LOG_LEVEL)
         _start_time = datetime.datetime.now()
 
-        version = data.get_info()['version']
+        version = data.get_version()
         logging.info(f'abibas v{version}. Light controllers MODBUS simulator')
 
         while run():
@@ -458,14 +339,12 @@ if __name__ == '__main__':
                 else:
                     handle_client(instance, c['Model'])
             sleep(5)
-            calculate_freq(5, datetime.datetime.now())
 
         report()
-        report_freq()
         dispose()
 
-    except Exception as ex:
+    except Exception:
         exc = sys.exception()
         traceback.print_tb(exc.__traceback__, limit=3, file=sys.stdout)
-        logging.critical(f'An epic failure just has happened! ({ex})')
+        logging.critical('An epic failure just has happened!')
         dispose()

src/ui/controller_types.py

@@ -15,9 +15,7 @@ bp = Blueprint('controller_types', __name__)
 @bp.route('/types')
 def list():
     models = data.get_controller_types()
-    return render_template('controller_types/list.html',
-                           config=models,
-                           footer=data.get_info())
+    return render_template('controller_types/list.html', config=models)
 
 
 @bp.route('/types/add', methods=("GET", "POST"))
@@ -36,8 +34,7 @@ def add():
         return redirect(
             url_for('controller_types.edit', name=request.form['name']))
 
-    return render_template('controller_types/create.html',
-                           footer=data.get_info())
+    return render_template('controller_types/create.html')
 
 
 @bp.route('/types/<name>/edit', methods=("GET", "POST"))
@@ -68,9 +65,7 @@ def edit(name: str):
                 existing = _prepare_for_editing(existing['Name'])
                 break
 
-    return render_template('controller_types/update.html',
-                           data=existing,
-                           footer=data.get_info())
+    return render_template('controller_types/update.html', data=existing)
 
 
 @bp.route('/types/<name>/copy', methods=("GET", ))
@@ -99,7 +94,6 @@ def _find_model_by_name(name: str, models: dict = None) -> dict:
 
 def _prepare_for_editing(name: str):
     types = ['Input', 'Holding', 'Coil', 'Discrete']
-    variants = ['DropdownVariants', 'RawBinary', 'RawInteger']
     exists = _find_model_by_name(name)
     if not exists:
         return None
@@ -110,7 +104,6 @@ def _prepare_for_editing(name: str):
                 processed.append(f'{k}={v}')
         reg['PossibleValues'] = '\n'.join(processed)
     exists['RegTypes'] = types
-    exists['DisplayVariants'] = variants
     return exists
 
 
@@ -120,10 +113,7 @@ def _save_model(model: dict, current_name: str):
     del model['RegTypes']
     _remove_reg(model, '')
     for r in model['Registers']:
-        if 'PossibleValues' in r:
         r['PossibleValues'] = _str_to_possible(r['PossibleValues'])
-        else:
-            r['PossibleValues'] = []
     models.insert(0, model)
     data.set_controller_types(models)
 
@@ -156,8 +146,6 @@ def _update_reg(model: dict, reg_name: str, form: dict):
             updated['CanOverride'] = v == 'True'
         elif k == f'{reg_name}-type':
             updated['Type'] = v
-        elif k == f'{reg_name}-display-as':
-            updated['DisplayAs'] = v
         elif k == f'{reg_name}-possible':
             updated['PossibleValues'] = _str_to_possible(v)
     _remove_reg(model, reg_name)

src/ui/controllers.py

@@ -1,6 +1,5 @@
 """Setting up light controllers and monitoring their state"""
 
-import re
 import data
 from flask import (Blueprint, flash, g, redirect, render_template, request,
                    url_for)
@@ -11,10 +10,8 @@ bp = Blueprint('controllers', __name__)
 
 @bp.route('/', methods=('GET', ))
 def list():
-    lst = get_list()
-    return render_template('controllers/index.html',
-                           config=lst,
-                           footer=data.get_info())
+    data = get_list()
+    return render_template('controllers/index.html', config=data)
 
 
 @bp.route('/override', methods=('POST', ))
@@ -30,22 +27,13 @@ def update():
         if k == 'enabled':
             enabled = v
             continue
-        registers.append({'Name': k, 'Value': v})
+        registers.append({'Name': k, 'Value': int(v)})
 
     controllers = data.get_controllers()
     existing = _find(port, controllers)
     if existing is None:
         abort(404)
 
-    current_regs = data.get_controller_registers(existing['Port'])
-    for r in registers:
-        reg = next(filter(lambda x: x['Name'] == r['Name'], current_regs), r)
-        display_as = reg.get('DisplayAs', 'DropdownVariants')
-        if display_as == 'RawBinary' and _is_binary_str(r['Value']):
-            r['Value'] = int(r['Value'], 2)
-        else:
-            r['Value'] = int(r['Value'])
-
     data.set_controller_registers(existing['Type'], existing['Port'],
                                   registers)
     enabled = enabled == 'True'
@@ -101,31 +89,15 @@ def add():
             data.set_controllers(controllers)
             return redirect(url_for('controllers.list'))
 
-    return render_template('controllers/create.html',
-                           types=types,
-                           footer=data.get_info())
+    return render_template('controllers/create.html', types=types)
 
 
-def _get_current_value(reg: dict) -> str:
-    if 'DisplayAs' in reg and reg['DisplayAs'] == 'RawBinary':
-        return f'{reg["CurrentValue"]:b}'.zfill(16)
-    else:
-        return reg['CurrentValue']
-
-
-def _get_value_name(reg: dict) -> str:
-    value = reg['CurrentValue']
-    display_as = reg.get('DisplayAs', 'DropdownVariants')
-    if display_as == 'RawBinary':
-        return value
-    elif display_as == 'RawInteger':
-        return ''
-    else:
-        for possible in reg['PossibleValues']:
+def _get_value_name(value: int, possibleValues: list) -> str:
+    for possible in possibleValues:
         for k, v in possible.items():
             if v == value:
                 return k
-        return 'Undefined'
+    return 'Unknown'
 
 
 def get_list() -> dict:
@@ -141,9 +113,9 @@ def get_list() -> dict:
                 'Name':
                 reg['Name'],
                 'Value':
-                _get_current_value(reg),
+                reg['CurrentValue'],
                 'ValueStr':
-                _get_value_name(reg),
+                _get_value_name(reg['CurrentValue'], reg['PossibleValues']),
                 'Address':
                 reg['Address'],
                 'Type':
@@ -151,9 +123,7 @@ def get_list() -> dict:
                 'PossibleValues':
                 reg['PossibleValues'],
                 'CanOverride':
-                reg['CanOverride'],
-                'DisplayAs':
-                reg.get('DisplayAs', 'DropdownVariants')
+                reg['CanOverride']
             })
 
     return controllers
@@ -164,8 +134,3 @@ def _to_status(enabled: bool) -> str:
         return 'Yes'
     else:
         return 'No'
-
-
-def _is_binary_str(s: str) -> bool:
-    pattern = r'^[01]+$'
-    return re.match(pattern, s) is not None

src/ui/static/style.css

@@ -1,12 +1,8 @@
 html {
   font-family: sans-serif;
   background: #eee;
-}
-
-body {
-  min-height: 100vh;
-  display: flex;
-  flex-direction: column;
+  padding: 1rem;
+  margin: 0.5rem;
 }
 
 nav {
@@ -190,25 +186,3 @@ label {
     flex-basis: 100%; /* Take full width of the container */
   }
 }
-
-footer {
-    margin-top: auto;
-    height: 2rem;
-    text-align: center;
-    background-color: #e5bdf1;
-    padding: 0.5rem 0.5rem;
-    opacity: 0.50;
-}
-
-footer p {
-    margin: 0;
-}
-
-.grey-box {
-    background-color: #d0d0d7;
-    padding: 0.2rem;
-    border: 1px solid #ccc;
-    display: inline-block;
-    font-size: 0.7rem;
-    border-radius: 0.3rem
-}

src/ui/templates/base.html

@@ -5,7 +5,6 @@
     <title>{% block title %}{% endblock %}</title>
     <link rel="stylesheet" href="{{ url_for('static', filename='style.css') }}">
   </head>
-
 <nav>
   <h1>Abibas (Automated Beacon Interface for Budget-friendly Aviation Safety)</h1>
   <ul>
@@ -23,7 +22,4 @@
     {% block content %}
     {% endblock %}
 </section>
-<footer>
-        <p>Version: {{ footer['version'] }} ⊚ Author: {{ footer['author'] }} ⊚ License: {{ footer['license'] }} </p>
-  </footer>
 </html>

src/ui/templates/controller_types/create-back.html

@@ -0,0 +1,27 @@
+{% extends 'base.html' %}
+
+{% block header %}
+  <h1>{% block title %}New Controller Type{% endblock %}</h1>
+{% endblock %}
+
+{% block content %}
+  <form method="post">
+    <div>
+      <div><label for="name">Name:</label></div>
+      <div><input name="name" id="name" value="{{ request.form['name'] }}" required></div>
+    </div>
+<br/>
+    <div>
+      <div><label for="runAs">Run As:</label></div>
+      <div><select id="runAs" name="runAs">
+      <option value="Server">Server</option>
+      <option value="Client">Client</option>
+      </select></div>
+    </div>
+<br/>
+    <div>
+      <input type="submit" value="Add">
+    </div>
+  </form>
+  <p>Note: once new type is created, you can Edit the type to add necessary registers. Changing "Run As" option will not be possible</p>
+{% endblock %}

src/ui/templates/controller_types/list.html

@@ -19,9 +19,7 @@
       <div class="setting-wide">
         <div><label for="reg-{{ reg['Name'] }}">{{ reg['Type'] }} {{ reg['Address'] }} ({{ reg['Name'] }}):</label></div>
 
-      <div>
-        {% if reg['PossibleValues'] %}
-      <select id="reg-{{ reg['Name'] }}" name="{{ reg['Name'] }}">
+       <div><select id="reg-{{ reg['Name'] }}" name="{{ reg['Name'] }}">
           {% for option in reg['PossibleValues'] %}
           {% for key, value in option.items() %}
 
@@ -29,14 +27,7 @@
 
           {% endfor %}
           {% endfor %}
-      </select>
-
-      {% else %}
-
-      <div class="grey-box">{{ reg['DisplayAs'] }}</div>
-
-      {% endif %}
-      </div>
+        </select></div>
       </div>
 
       {% endfor %}

src/ui/templates/controller_types/update.html

@@ -61,17 +61,10 @@
       <div><label for="{{ reg['Name'] }}-address">Address:</label></div>
       <div><input name="{{ reg['Name'] }}-address" id="{{ reg['Name'] }}-address" value="{{ request.form[reg['Name'] ~ '-address'] or reg['Address'] }}" placeholder="12345" required></div>
     </div>
-    <div>
-      <div><label for="{{ reg['Name'] }}-display-as">Display value as:</label></div>
-      <div><select id="{{ reg['Name'] }}-display-as" name="{{ reg['Name'] }}-display-as">
-          {% for variant in data['DisplayVariants'] %}
-          <option value={{ variant }} {% if variant == reg['DisplayAs'] %}selected="selected"{% endif %}>{{ variant }}</option>
-          {% endfor %}
-      </select></div>
-    </div>
+
     <div>
       <div><label for="{{ reg['Name'] }}-possible">Possible values:</label></div>
-      <div><textarea name="{{ reg['Name'] }}-possible" id="{{ reg['Name'] }}-possible" rows="6" cols="28" placeholder="ValueNameOne=14&#13;ValueNameTwo=3" {% if reg['DisplayAs'] == 'DropdownVariants' %} required {% else %} disabled {% endif %} >{{ request.form[reg['Name'] ~ '-possible'] or reg['PossibleValues'] }}</textarea></div>
+      <div><textarea name="{{ reg['Name'] }}-possible" id="{{ reg['Name'] }}-possible" rows="6" cols="28" placeholder="ValueNameOne=14&#13;ValueNameTwo=3" required>{{ request.form[reg['Name'] ~ '-possible'] or reg['PossibleValues'] }}</textarea></div>
     </div>
       </article>
     {% endfor %}

src/ui/templates/controllers/index.html

@@ -25,7 +25,7 @@
 
       <div class="setting">
         <label {% if not reg['CanOverride'] %} class="red" {% endif %}>{{ reg['Name'] }} ({{ reg['Type'] }}):</label>
-        <div>[{{ '%05d' % reg['Address'] }}] ➡ {{ reg['Value'] }} {% if reg['ValueStr'] != '' %}({{ reg['ValueStr'] }}){% endif %}
+        <div>[{{ '%05d' % reg['Address'] }}] ➡ {{ reg['Value'] }} ({{ reg['ValueStr'] }})
         </div>
       </div>
 
@@ -49,25 +49,23 @@
         </select></div>
         </div>
 
+
         {% for reg in controller['Registers'] %}
         {% if reg['CanOverride'] %}
 
       <div class="setting-wide">
         <div><label for="reg-{{ reg['Name'] }}">{{ reg['Name'] }} value:</label></div>
-
-        {% if not reg['DisplayAs'] or reg['DisplayAs'] == 'DropdownVariants' %}
-
          <div><select id="reg-{{ reg['Name'] }}" name="{{ reg['Name'] }}">
           {% for option in reg['PossibleValues'] %}
           {% for key, value in option.items() %}
-          <option value="{{ value }}" {% if value == reg['Value'] %}selected{% endif %}>{{ key }} ({{ value }})</option>
+          {% if value == reg['Value'] %}
+          <option value="{{ value }}" selected>{{ key }} ({{ value }})</option>
+          {% else %}
+          <option value="{{ value }}">{{ key }} ({{ value }})</option>
+          {% endif %}
           {% endfor %}
           {% endfor %}
         </select></div>
-
-         {% else %}
-         <div><input id="reg-{{ reg['Name'] }}" name="{{ reg['Name'] }}" value="{{ reg['Value'] }}" required></div>
-         {% endif %}
       </div>
 
        {% endif %}