simulation: extra_params might be a list

- still accept comma separated string
- remove legacy naming '.extra_params'

Change-Id: I497cf7722d0b39dd31c516383449a4cc4e7dcb7d
Reviewed-on: https://forge.frm2.tum.de/review/c/secop/frappy/+/32968
Tested-by: Jenkins Automated Tests <pedersen+jenkins@frm2.tum.de>
Reviewed-by: Alexander Zaft <a.zaft@fz-juelich.de>
Reviewed-by: Markus Zolliker <markus.zolliker@psi.ch>

frappy/simulation.py

@@ -31,12 +31,13 @@ from frappy.modules import Drivable, Module, Parameter, Readable, Writable, Comm
 
 class SimBase:
     def __new__(cls, devname, logger, cfgdict, dispatcher):
-        extra_params = cfgdict.pop('extra_params', '') or cfgdict.pop('.extra_params', '')
+        extra_params = cfgdict.pop('extra_params', '')['value']
+        if isinstance(extra_params, str):
+            extra_params = [v.strip() for v in extra_params.split(',')]
         attrs = {}
         if extra_params:
-            for k in extra_params['value'].split(','):
-                k = k.strip()
-                attrs[k] = Parameter(f'extra_param: {k.strip()}',
+            for k in extra_params:
+                attrs[k] = Parameter(f'extra_param: {k}',
                                      datatype=FloatRange(),
                                      default=0.0)