Go to Post When teams work together to design things rather than going it alone, you can really come up with magnificent machines. - JesseK [more]
Home
Go Back   Chief Delphi > Technical > Programming > NI LabVIEW
CD-Media   CD-Spy  
portal register members calendar search Today's Posts Mark Forums Read FAQ rules

 
Reply
 
Thread Tools Rate Thread Display Modes
  #1   Spotlight this post!  
Unread 18-01-2012, 01:09
SuperS_5's Avatar
SuperS_5 SuperS_5 is offline
[Certified LabVIEW Developer]
FRC #1219
 
Join Date: Dec 2010
Rookie Year: 2010
Location: Canada
Posts: 140
SuperS_5 will become famous soon enoughSuperS_5 will become famous soon enough
Bug in Compress state in WPI library

I think I found an issue with this VI. Another thread peaked my interest in the compressor functions, and browsing through them, I found that this functional global seems to support multiple compressors at first sight.

...\vi.lib\Rock Robotics\WPI\Compressor\WPI_Compressor Global.vi

Unfortunately, this VI seems to overwrite the "compressor state out" with potentially the wrong data. It also doesn't check if the same compressor ref is already added to the stack.
I am sure most teams would only use one compressor anyways, if any. Not even sure if the rules allow for more. However, if a second compressor is used, for example in the off-season, the API will fail.
__________________
Mike B
Reply With Quote
  #2   Spotlight this post!  
Unread 18-01-2012, 08:24
Greg McKaskle Greg McKaskle is offline
Registered User
FRC #2468 (Team NI & Appreciate)
 
Join Date: Apr 2008
Rookie Year: 2008
Location: Austin, TX
Posts: 4,751
Greg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond reputeGreg McKaskle has a reputation beyond repute
Re: Bug in Compress state in WPI library

I see it now. The loop needs to terminate when the ref is handled. I'll add a tracker.

Greg McKaskle
Reply With Quote
  #3   Spotlight this post!  
Unread 18-01-2012, 13:21
SuperS_5's Avatar
SuperS_5 SuperS_5 is offline
[Certified LabVIEW Developer]
FRC #1219
 
Join Date: Dec 2010
Rookie Year: 2010
Location: Canada
Posts: 140
SuperS_5 will become famous soon enoughSuperS_5 will become famous soon enough
Re: Bug in Compress state in WPI library

That is what I have added to my local copy.
__________________
Mike B
Reply With Quote
Reply


Thread Tools
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT -5. The time now is 11:35.

The Chief Delphi Forums are sponsored by Innovation First International, Inc.


Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.
Copyright © Chief Delphi