Skip to content

Commit f1281d5

Browse files
committed
add minutes from November meeting
1 parent 45bd8f3 commit f1281d5

File tree

1 file changed

+198
-0
lines changed

1 file changed

+198
-0
lines changed

minutes/2025-11-07_vidconf.rst

Lines changed: 198 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,198 @@
1+
meeting 2025-09-17 (ZOOM)
2+
@@@@@@@@@@@@@@@@@@@@@@@@@
3+
4+
.. sidebar:: participants
5+
6+
* Georg Brandl
7+
* Peter Braun
8+
* Alexander Zaft
9+
* Markus Zolliker
10+
* Klaus Kiefer
11+
* Enrico Faulhaber
12+
* Niklas Ekström
13+
* Bastian Klemke
14+
* Zeus Castillo
15+
* Eddy Lelièvre-Berna
16+
17+
.. contents:: Agenda
18+
:local:
19+
:depth: 3
20+
21+
1) presentation of PLC implementation @ILL
22+
==========================================
23+
24+
Zeus Castillo presents his implemention of SECoP @ ILL.
25+
He created an open source Codesys library to be re-used by others.
26+
Several interesting implementation details are presented.
27+
Also, a live demo was shown and its details were explained.
28+
Operation via a simple text client and via the frappy-client worked flawlessly.
29+
30+
Several questions by Georg and Klaus got answered.
31+
Especially, the way the descriptive data is created raised some questions.
32+
33+
Zeus explained that the format of the configuration file is preliminary and
34+
that the config file for his code generator just look very similiar to the
35+
descriptive data, as it was used as a strating point.
36+
37+
Georg asks about some details of JSON parsing/generation.
38+
Zeus explained his solution.
39+
40+
Eddy, unfortunately, has to leave.
41+
42+
Alexander queries Zeus about how easy to find and understandable the current SECoP specification
43+
was for him. Zeus reported, that it was easy to find and (after reading it several times)
44+
also easy to understand. Also, he contacted Markus about unclear things, which were quickly resolved.
45+
The only difficult part was the example of the 'check' message, as this was the only occurence of a triple-valued device.
46+
47+
Georg queries about whether everything is run within the same task and which cycle times are possible.
48+
Zeus reported the current solution to be singletask, but it may be split into multiple tasks
49+
with distinct priorities so that SECoP communications doesn't block higher priority tasks.
50+
cycle times around 20ms to 50ms are currently used and work fine.
51+
52+
Klaus wonders about how the handshaking define in the SECoP spec is implemented.
53+
Zeus shows and explains several implementation details, answering this question as well.
54+
55+
Enno wonders about why the implementation on a PLC was choosen.
56+
A small discussion, including fear of having to update, essentially ends with the coclusion
57+
that there are some subtle differences between facilities which faviour one, or the other solution.
58+
59+
60+
2) approval of previous minutes (2025-10-14/15)
61+
===============================================
62+
63+
approved.
64+
65+
66+
3) discussion process on github
67+
===============================
68+
69+
XXX: may need to rephrase the topic of this section
70+
71+
Georg points out, that while on https://www.github.com/SampleEnvironment/SECoP ,
72+
just 'click' on the 'eye' and select 'all-notifications'.
73+
74+
Georg proposes to communicate the possibility to comment on recent activities on the SECoP spec
75+
via the ISSE newsletter or website.
76+
77+
Georg presents his work on splitting several pages of the current specification
78+
into multiple smaller sections.
79+
E.g. There are now individual pages per message and an index.
80+
A section showing the differences between distinct versions aof the specification is also included.
81+
82+
The presentation is welcome, the work is greatly appreciated.
83+
84+
Zeus points out that, while this new way to structure the pages is much easier to find
85+
relevant information, more examples e.g. when to use which errorclass may further improve
86+
the usefulness of the specification.
87+
88+
Klaus points out that we have, by now, mor then 3 defined interface_classes.
89+
There is the Communicator and the Acquisition classes.
90+
91+
Agreement to activate this version of the specification.
92+
93+
Zeus leaves.
94+
95+
96+
4) Acquisition RFC
97+
==================
98+
99+
Georg shows the current state.
100+
Markus proposes to just merge this now.
101+
Agreement on merging this.
102+
103+
104+
5) YAML (RFC 2/3) and Systems (RFC 4)
105+
=====================================
106+
107+
Georg points out that due to the restructuring of the specifiaction,
108+
some parts of the actual pull-request may not work anymore and need rework.
109+
110+
Markus thinks we don't need to hurry, but it should be done soon.
111+
klaus points out, that next april the specification has its 10(th) birthday, so
112+
it would be nice to be finished with this until then.
113+
114+
Georg and Markus discuss details within RFC002 about checking the validity of the
115+
ParameterPostfixes.
116+
117+
Georg will rework the relevant pieces of the specification to add the systems and yaml files.
118+
119+
120+
6) revisiting old issues
121+
========================
122+
123+
Issue 066 force re-connect
124+
--------------------------
125+
126+
seems to have been forgotten to include in the spec.
127+
Agreement to move to the spec.
128+
129+
130+
Issue 67 pid control parameter
131+
------------------------------
132+
133+
Issue got stuck, as there was no agreement.
134+
Klaus points out that this need to be reconsiderated when we write a system for a temperature controller.
135+
136+
Agreement to leave it as is and solve the issue when writing the systems.
137+
138+
139+
Issue 69 optional structs by default
140+
------------------------------------
141+
142+
A discussion about the usefulness of optional parts of a struct starts.
143+
The changed reply, however, should always include all struct members.
144+
145+
It turns out, that optional struct members are part of the spec already,
146+
just that the issue would introduce a breaking change.
147+
148+
Consensus seems to leave this at the current state and close the Issue.
149+
An example seems to be missing in the spec.
150+
151+
152+
Issue 077 prefixes
153+
------------------
154+
155+
It was already agreed (2023-01-16) that we use postfixes instead of prefixes.
156+
Peter is going to write a pull-request.
157+
158+
Agreed postfixes are '_enabled', '_min', '_max', and '_limits'.
159+
160+
161+
Issue 078 Interacting modules (use case power supplies)
162+
-------------------------------------------------------
163+
164+
Markus proposes to treat it as a system.
165+
166+
167+
- Markus would leave it out of 2.0
168+
- Georg says it would be good to have at least one system as an example for 2.0
169+
- Klaus agrees, but mentions it does not have to be in the spec for 2.0 at first
170+
171+
Klaus and Ennno have to leave.
172+
173+
Converting Issues to RFC's
174+
--------------------------
175+
176+
Georg proposes to convert at least some issues to RFC format. Alexander suggests keeping the issue namespace separate, RFC-9xx may be confusing.
177+
Proposal from Markus and Georg: issues should keep their numbers, renumber the existing RFCs to 101 to 108 and continue from there.
178+
179+
A.o.B.
180+
======
181+
182+
Klaus gives some information about the ISSE meeting:
183+
184+
Possible dates for the committee meeting will be:
185+
186+
- 28.11.2025
187+
- 01.12.2025
188+
189+
Klaus would like a meeting before that.
190+
191+
Date of next Meeting
192+
====================
193+
194+
There will be a short meeting on 26.11.2025 at 09:00 (alternative: 25.11.2025 09:00). Clarify with Enno.
195+
196+
..
197+
-- closed at 15:43 --
198+

0 commit comments

Comments
 (0)