In line or Sub-VI ?

Question

I have created a VI to do the following :
1. Get raw data from 4 AI channels once every 50ms.
2. Accumulate these in a 2D array as long as a cycle lasts.
3. At the end of the cycle find the mean of all four arrays and check if they are In-Bound and save the result in a BooleanArray.
4. Repeat steps 1 to 3 for three iterations.
5. At the end of the third iteration, find out if each of the channles have been within bound based on the boolean array and declare a Verdict.
( Verdict is FAIL if all the three iterations of one or more channels are FALSE )
The VI is sfucntional and attached. The various indicators help to track the performance and not need in the final version. I only need the controls for CycleCount,  NoOf Channels, LoLimit array and HiLimitArray and the final Verdict information.
Question is :
1. How to convert this into a sub VI  so that I can call this from inside of a 50ms timed loop that the has the DAQMx AI read function?
2. Or is it easy to just copy this code inside of a TimedLoop which my application anyway has ?
Thanks
Raghunathan
Raghunathan
LV2012 to Automate Hydraulic Test rigs.
Attachments:
ParamOK_ArrayIn.vi ‏113 KB

Answer 1

Raghunathan wrote:
2. Or is it easy to just copy this code inside of a TimedLoop which my application anyway has ?
First of all, your VI is overly complicated. You need two shift registers just to keep track if indices because you use "insert into array" to appen a row to an existing array. You don't need those if you use "built array".
(Actually, if performance is an issue, you should allocate a fixed size array and replace columns as you go. In this case you would need these shift registers again )
The final code is almost nothing, and I don't think it is worth to create a subVI for it. Just put it inline in the main VI.
You are doing things again way too complicated! Here are some examples:
Your code skips an acquisition set whenever it does the calculation (your new data array doe not enter the case!). You should use all data!
You do an incredible song and dance to duplicate what we already have in "AND array elements" and "Or array elements".
You don't need the "NoOfChannels" control, because the number is given by the array dimension already.
The attached is a quick attempt to solve some of these issues. Now it seems to operate correctly. See it if makes sense to you. The code works well at 50ms loop time.
LabVIEW Champion . Do more with less code and in less time .
Attachments:
ParamOK_ArrayInmod.vi ‏104 KB