Go to Post So we dont need a brain to run the minibot? - TheGuyz [more]
Home
Go Back   Chief Delphi > Technical > Programming > C/C++
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 03-06-2013, 12:51 PM
virtuald's Avatar
virtuald virtuald is offline
RobotPy Guy
AKA: Dustin Spicuzza
FRC #1418 (), FRC #1973, FRC #4796, FRC #6367 ()
Team Role: Mentor
 
Join Date: Dec 2008
Rookie Year: 2003
Location: Boston, MA
Posts: 1,033
virtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant futurevirtuald has a brilliant future
Re: New REQUIRED version of C++ libraries

Quote:
Originally Posted by Mike Copioli View Post
Brad,

Thank you for this update.

What is the observed robot behavior that this fixes?
A diff can be seen here. Looks like the big changes are disabling TCP_NODELAY, requiring a newer version of the Jaguar firmware, and fixing the way it searches for entries.
__________________
Maintainer of RobotPy - Python for FRC
Creator of pyfrc (Robot Simulator + utilities for Python) and pynetworktables/pynetworktables2js (NetworkTables for Python & Javascript)

2017 Season: Teams #1973, #4796, #6369
Team #1418 (remote mentor): Newton Quarterfinalists, 2016 Chesapeake District Champion, 2x Innovation in Control award, 2x district event winner
Team #1418: 2015 DC Regional Innovation In Control Award, #2 seed; 2014 VA Industrial Design Award; 2014 Finalists in DC & VA
Team #2423: 2012 & 2013 Boston Regional Innovation in Control Award


Resources: FIRSTWiki (relaunched!) | My Software Stuff
Reply With Quote
  #2   Spotlight this post!  
Unread 03-07-2013, 03:47 PM
taichichuan's Avatar
taichichuan taichichuan is offline
Software Mentor
AKA: Mike Anderson
FRC #0116 (Epsilon Delta)
Team Role: Mentor
 
Join Date: Feb 2010
Rookie Year: 2010
Location: Herndon, VA
Posts: 328
taichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud of
Send a message via AIM to taichichuan
Re: New REQUIRED version of C++ libraries

Quote:
Originally Posted by virtuald View Post
A diff can be seen here. Looks like the big changes are disabling TCP_NODELAY, requiring a newer version of the Jaguar firmware, and fixing the way it searches for entries.
Hmm... Disabling TCP_NODELAY. That will add several ms of latency to the link. Was there a problem in polling the socket connection that required this? Just curious because in almost 30 years of working with VxWorks, you typically want to *add* TCP_NODELAY to reduce communicaitons latency rather than remove it.

TIA,

Mike
Reply With Quote
  #3   Spotlight this post!  
Unread 03-10-2013, 10:02 PM
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,748
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: New REQUIRED version of C++ libraries

I haven't seen the code, but my understanding is that at last some of the writes were done a byte at a time -- likely with the iostream ++ syntax. That plus the no-delay resulted in lots of packets that made the router unhappy.

I agree with your comment about no-delay. I'd expect the latency to be around as much as 100ms.

Greg McKaskle
Reply With Quote
  #4   Spotlight this post!  
Unread 03-10-2013, 10:32 PM
taichichuan's Avatar
taichichuan taichichuan is offline
Software Mentor
AKA: Mike Anderson
FRC #0116 (Epsilon Delta)
Team Role: Mentor
 
Join Date: Feb 2010
Rookie Year: 2010
Location: Herndon, VA
Posts: 328
taichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud oftaichichuan has much to be proud of
Send a message via AIM to taichichuan
Re: New REQUIRED version of C++ libraries

Quote:
Originally Posted by Greg McKaskle View Post
I haven't seen the code, but my understanding is that at last some of the writes were done a byte at a time -- likely with the iostream ++ syntax. That plus the no-delay resulted in lots of packets that made the router unhappy.

I agree with your comment about no-delay. I'd expect the latency to be around as much as 100ms.

Greg McKaskle
Hi Greg,

Yeah, I could believe that. Given the number of packets in normal driver station comms, I could see that there's likely a fair amount of 802.3x flow control packets if the switches are being overrun. However, this sounds like a hack. VxWorks is a very deterministic O/S. If it's resorting to very short payloads, it's likely because there's a resource starvation going on someplace and this is a quick hack to get past the problem. I've seen multiple places in WPILib that look like desktop folks trying to understand operating in real-time -- unsuccessfully. But, they keep introducing additional layers of abstraction hoping to make things easier. However, abstraction layers have a price. I suspect that this small payload issue is one of those prices.

How much longer are we using this control system? There are many ways to improve performance on the 'bot without losing the 3 language support. I've been a long time user and developer on VxWorks. But the reality is that WRS, really isn't doing much to promote the use of VxWorks on the 'bot. They're really missing an opportunity, IMHO.

Thanks,

Mike
Reply With Quote
  #5   Spotlight this post!  
Unread 03-18-2013, 08:34 AM
sircedric4's Avatar
sircedric4 sircedric4 is offline
Registered User
AKA: Darren
no team (The SS Prometheus)
Team Role: Mentor
 
Join Date: Jan 2008
Rookie Year: 2006
Location: Lousiana
Posts: 245
sircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond repute
Re: New REQUIRED version of C++ libraries

I am getting everything ready for our regional this week and I have installed this required update and rebuilt my code.

Is there some way I can confirm that I have the workbench update installed so I don't have to panic on Thursday? Is there a version number somewhere I can check?

In the past I sometimes had trouble with the self installers working and I just want to confirm I am ready to go. How would the inspectors check it?
Reply With Quote
  #6   Spotlight this post!  
Unread 03-18-2013, 09:50 AM
Jefferson Jefferson is offline
Registered User
AKA: Jeff Clements
FRC #0016 (Bomb Squad)
Team Role: Mentor
 
Join Date: Jan 2011
Rookie Year: 2010
Location: Mountain Home, AR
Posts: 257
Jefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant futureJefferson has a brilliant future
Re: New REQUIRED version of C++ libraries

Quote:
Originally Posted by sircedric4 View Post
Is there some way I can confirm that I have the workbench update installed so I don't have to panic on Thursday? Is there a version number somewhere I can check?... How would the inspectors check it?
You can check by navigating to:
C:\WindRiver\WPILib
You should have a .zip file in that folder of the format: WPILibC++SourceYYYYMMDDrevXXXX. The XXXX should be 3622. I have not been to an event since this was released, but I believe the inspectors will check this same way.
Reply With Quote
  #7   Spotlight this post!  
Unread 03-18-2013, 11:52 AM
sircedric4's Avatar
sircedric4 sircedric4 is offline
Registered User
AKA: Darren
no team (The SS Prometheus)
Team Role: Mentor
 
Join Date: Jan 2008
Rookie Year: 2006
Location: Lousiana
Posts: 245
sircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond reputesircedric4 has a reputation beyond repute
Re: New REQUIRED version of C++ libraries

Quote:
Originally Posted by Jefferson View Post
You can check by navigating to:
C:\WindRiver\WPILib
You should have a .zip file in that folder of the format: WPILibC++SourceYYYYMMDDrevXXXX. The XXXX should be 3622. I have not been to an event since this was released, but I believe the inspectors will check this same way.
Thank you, I have confirmed my computer is up to date.
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 04:04 AM.

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