I looked at your code. The wires are straight enough, it still fits on a screen and the flow left->right is respected. It is easy to read, that's important.
For the what is achieved it does not require much more. If you provide more information on the final purpose we could have a better idea of the complexity and suggest an architecture for example.
I still give you some remarks especially useful if the program continues to grow:
- I don't understand the purpose of the display loop. If you want to forbid the user to rotate the model then use CameraController.Type=None.
- No need for two while loops if they have the same wait time
- Reset the boolean controls when stopping the program
- You could start to use sub-VI's for the calculation of the angle and reuse this VI for X and Y positions. There would be two inputs (the control and the reset position)
- At some point you will need to handle the errors. It is great for the flow of data. Even in case of an error it gives the program a chance to close the references, the files, reset the controls, etc. And if the final user is not a developer, he probably does not want to see a LabVIEW error with incomprehensible words.
- Please use comments. Your program is small and easy to understand now that you are working on it. But even in a few weeks, comments could be useful. It saves time
Yddet