LabVIEW
Read 3 serial port from Arduino in labview
Hi amkamyab,
please don't create duplicate threads for the very same topic.
Especially when you don't even attach a down-converted version of your VI!
Edit after your last message:
To downconvert a VI you go open the File menu -> Save for previous…
what do you think about it. should i need to change my code or do some change in labview or arduino.
Yes, surely you NEED TO change your code!
1. Don't use BytesAtPort! Just wire a constant larger than the expected message size! (You set VISASerialPortInit to wait for TermChars!)
2. There is no error handling at all right now!
3. Separate each communication (COM port) in its own loop.
4. Use more descriptive labels for your controls.
5. You don't need to open the COM port, VISASerialPortInit is sufficient.
Hi amkamyab,
what kind of messages do you want to receive from your Arduino(s)? How are they formatted?
Please provide more information!
You also don't show how your VI looks like right now…
At VISASerialPortInit you enabled the TermChar and set LF as TermChar, so I guess your messages are send using a Serial.PrintLN command at the Arduino. (If this is not the case you need to use a better setup at initialising the COM ports and adapt your reading loop.)
Hi amkamyab,
OMG!
What is this monster of a VI made to do?
Why do you need SO MUCH duplicated code?
Why do you need to convert arrays to clusters so often - just to get 2 boolean values???
Why is there so much dead code in yur VI? (All those HighResolutionTimer calls!)
There still is no error handling.
There still is no explanation of the message and its formatting.
You seems to receive a value from 0 to 20 from your Arduino. Then you add a boolean TRUE to an array, depending on the value you received. WHY don't you use just ONE array to store all data, with the received value used to index an element/row in the buffer???
What kind of data (structure) do you need?
Edit after noticing that Arduino code:
The only relevant line of code is this:
Serial.write(buffer2[i] );
"buffer2" is initialized as array of bytes, so that Serial.write is sending just one byte (aka U8).
- You need to disable the TermChar at the VISASerialPortInit function!
- You need to read exactly one byte (aka char). No need for BytesAtPort! (Well, you could read more than one byte, but then you need to handle the received byte array in a loop.)
- You need to convert the received string to an U8 array instead of using FractStringToNumber!
-
