Jump to content

Peer review - Data Acquisition read and write .vi


Recommended Posts

Hey all,

Just looking to get some online peer review since it's non-existant at my work.

I developed this program to do some data acquisition using a fleet of around 15 DAQ suitcases. The cases are loaded with cDAQ-9174's with a 9203 current module in them. I'm using a couple of pressure transducers, a couple of RTD's and a couple of differential pressure transducers per case, all run by USB back to an Acer Netbook that does the thinking.

It's pretty straight forward right now, but I'm open to any ideas for simplifacation or farkles.

Thanks in advance,

-Ian

SuitcaseTopLevel.vi

Flow Formula - Dwyer Averaging Pitot.vi

Link to comment

I do not think this VI will work as you have intended.

You have a Queue owning the consumer loop at the bottom of the VI, however, you have a state selector which responds to a shift registered enum.

This will create a situation where you will display the data you write, but you are junking 1/4 of the data each time. That case structure cannot enter your desired state until the signal is updated by your producer loop.

Perhaps you could place your dequeue operation inside your display state to create a lossless display/write cycle.

~Jon

Link to comment
Perhaps you could place your dequeue operation inside your display state to create a lossless display/write cycle.

Very good point that I hadn't even considered. I was unknowingly throwing away a bunch of update data, probably never noticed in this application since the update screen will only be there to act as a sanity check during install and the case will remain closed most of the time with it's screen off, but good idea for future programs where it could cause issues. Thanks for the heads up.

-Ian

Link to comment

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.