chrisger,
A few points: The cluster should be inside the event structure (or at least inside the while loop), else the latch action booleans never reset because they are never read by the diagram. Also the stop button should be latch action and must have the terminal inside (no local variables needed now)
I am not sure how you ensure that the local variables on the left are written at the right point in time, because there is no data dependency to the rest of the code. Should they execute before or after the loop? You should probably ensure that things execute in a predictable order (Not that it probably matters much in this particular case, but in other situations this kind of coding can cause very unpredictable behavior and should be avoided.)
Property nodes are expensive so it does not make much sense to take a detour via the text labels and a multicase structure just to end up with a plain number at the end. 😉
All that's needed is the code alternative I proposed above. I have it now isolated for simplicity (see attached).