Remove your first while loop. You are initializing the serial port over and over again until you press OK. You only need to initialize once. Put all of the code before the loop. The loop should just delay for a few mS until the OK is pressed. You should always put a dely in a loop so as not to chew up 100% CPU time. Even a 0mS delay will allow the CPU time to do other things.
In the second loop, since you have an event structure, you don't need a delay. But you do need to wire in a value to the Timeout hourglass icon on the top left corner. I typically use 100. Again this is to prevent 100% CPU usage.
Do you understand data flow execution? This means that a loop or block of code or a function will not execute until all of its inputs are present. Your third loop will not start until you press the quit button because you have two wires going from the second loop to the third. These wires will only pass their data to the third loop when the second loop is over (pressing the quit button). When you press quit, the second loop terminates and data goes to the third loop. Now all of the third loop's inputs are present and it can execute. This is not what you want. Delete the wires between second and third loop. Then the third loop will execute from the beginning. The serial read control will tell it when to read and when not to read. You will need to pass the VISA Resource Name to the third loop by using a Local Variable. From your statement, it sounds like you already did this. Pass a no error condition into the third loop also. You can make one or use a local variable of error in.
In the third loop, you have the VISA Close inside the loop. This is not good. The VISA session will close before you read. Move this to after the third loop. Then it will close only after the third loop stops.
After you delete the wires between loops 2 and 3, you will have to merge errors from the second and third loops to make sure you catch all errors.
In each one of your loops, replace the VISA Session terminals and Error terminals with shift registers. This is good practice and is recommended by NI. With terminals, if there is an error on one loop iteration, it will get lost because the next iteration takes in the original no error condition from the left side. With a shift register, the error gets propogated to the next loop iteration. If your loop executes zero times (For Loops can do this), the output of the VISA terminal will be a null value and the VISA close will cause an error. With a shift register, even if the loop does not iterate at all, the VISA Session value on the input will be propogated to the output.
I think that is enough for now. Please make these changes and re-post your vi. You are making good progress.
