Hi gal,
please cleanup that VI before we start any further debugging!
There is so much Rube-Goldberg included…
Why do you need to use IndexArray twice to index the very same elements - right next to each other?
Why do you need to duplicate case structures 5 times to handle 5 array elements? Use an autoindexing loop to repeat a certain operation!
You don't need to wire constants at IndexArray to index elements starting from index 0…
Don't use BytesAtPort to read your serial port! Implement some error handling after reading the port…
What's the point of the sequence frame when reading the port?
What's the point of the outer FOR loop - especially when it is set to iterate just once!?
What's the point of shift registers when you don't use their data at the next iteration?
Why do you build an array of 5 elements based all on the very same single value??? Why do you need to evaluate that single value 5 times???
Much much more silly stuff…