08-01-2017 01:27 PM
Looking from your perspective I agree it has to be more clear. However one would mention nX1 or just 'n' for a 1 D array. Even though 1xn is supposed to be an array/ vector technically. I observed labview terminology where it considered to be a 2D array.
08-01-2017 01:36 PM - edited 08-01-2017 01:40 PM
You actually only have 1D array initially (and reshaping them to 2D and all the other Rube Goldberg stuff(!) does not change that.)
So all you need is implement the original suggestion.
And yes, also in your main VI, most of the reshaped 2D arrays should be 1D arrays! try to fix that!
08-01-2017 01:48 PM
And yes, looking at you DAQ-temp-Press VI, you really need to go gack to the drawing board and do some introductory LabVIEW tutorials. This thing is full of race conditions and greedy loops (why do you need that loop in the upper left??? Just to consume more battery/electricity and heat the room? 😮 What's up with all the vestigial one-iteration FOR loops? Why are there so many local variables and value property nodes?)
08-01-2017 01:53 PM
Altenbach
I got my answer and I accepted it.
I mentioned to you before there is a reason, why one would do things the way they do - probably because they have other vi's or future stuff they were planning to implement that you are not aware of. I do know I have a 1D array and I was intentionally converting to 2D (again no need to restate the obvious), it is visible in both of my main and test vi's.
If you think, I have to do the only way you suggested, thank you for the suggestion - but I will pass.
08-01-2017 02:01 PM
Why would you convert a 1-D array to a 2-D array to just do more steps to undo it?
Use the original 1-D arrays to append them. If you need those to be made into 2-D arrays for use elsewhere, then split the wires and do that.
08-01-2017 02:11 PM - edited 08-01-2017 02:50 PM
@looser_engineer wrote:
I mentioned to you before there is a reason, why one would do things the way they do - probably because they have other vi's or future stuff they were planning to implement that you are not aware of. I do know I have a 1D array and I was intentionally converting to 2D (again no need to restate the obvious), it is visible in both of my main and test vi's.
If you want to convert to a 2D array, all you need is insert another "built array" of height=1. All that song and dance with reshaping is just plain silly and nothing in your code would indicate that this is obvious and necessary. Almost nowhere in your code are 2D arrays needed, except when you build them into a 2D array for the graph and table, and a single "built array" would suffice there. (you can use reshape to trim/pad 1D arrays if needed).
I am very aware that your code is inefficient and overly complicated because I've been looking at LabVIEW code for over 20 years. Your overall design makes it much harder to expand in the future and since you are using 10x too much code, making the program even larger will make it unmanageable. Your program is a nightmare!
A greedy loop has no justification anywhere. Ever!
@looser_engineer wrote:
If you think, I have to do the only way you suggested, thank you for the suggestion - but I will pass.
I will strongly urge you to reconsider my advice. It is much harder to lose bad habits later. At this time you are not in a position to judge the quality of the various suggestion made here so you need to trust us. Attitude does not advance your skills. Maybe in a few month you could "upgrade" your username once it is no longer appropriate. 😄
08-07-2017 04:10 PM
Altenbach,
He/she named him/herself "Looser" (as in "not up-Tighter"), not "Loser" Engineer, but given the size of the on-shoulder Chip, I understand how you mis-understood ...
BS
08-07-2017 05:56 PM
I was going by the assumption that the spelling skills of the user correlated with the LabVIEW skills and extrapolated from there ... :D.
08-07-2017 08:37 PM
Dang -- you'd think, by now, I'd know better than to try to outwit Altenbach ... still have a ways to go!
BS