Go to Post Batteries are like clamps or money, you can never have enough. - IndySam [more]
Home
Go Back   Chief Delphi > Technical > Programming
CD-Media   CD-Spy  
portal register members calendar search Today's Posts Mark Forums Read FAQ rules

 
 
 
Thread Tools Rate Thread Display Modes
Prev Previous Post   Next Post Next
  #5   Spotlight this post!  
Unread 14-01-2005, 12:11
Mr. Lim Mr. Lim is offline
Registered User
AKA: Mr. Lim
no team
Team Role: Leadership
 
Join Date: Jan 2004
Rookie Year: 1998
Location: Toronto, Ontario
Posts: 1,125
Mr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond reputeMr. Lim has a reputation beyond repute
Re: CMUCam2 Camera Code - Are important parts commented out?

Quote:
Originally Posted by Mike Betts
Roger,

It makes sense that FIRST/IFI would comment out code so that it is safe for people to simply "run" on a robot without fully understanding function.
Although I agree with you, in this particular case it is a misrepresentation to publish and distribute a document like: "Start_Here-CMUcam2_fe.pdf" that gives absolutely no mention to having to uncomment code.

Quote:
I would suggest that you implement a piece at a time with appropriate debug printf statements to understand what each part does. Note that some code may require modification by you to be compatible with your system.

As an example, I would insure that the camera is giving good information to your controller before I attempted to let it guide the drive system of your robot.

Mike
We are receiving proper T-packets, with good x/y cam co-ordinates. FIRST/IFI already included a printf debug statement in this regard (from user_routines_DDT.c):
Code:
	// Put vision processing here, because we have a good frame!
	printf( "Got T packet %d %d %d %d servo: %d %d\r",cam.x,cam.y,cam.size,cam.conf,cam.pan_servo,cam.tilt_servo );
	if (cam.size > 0 && p1_sw_trig>0)	{			//Check to see if camera is tracking
		tracking = 1;								// if yes then set 'tracking' flag
		pan_position = cam.pan_servo;
	}
	else
		tracking = 0;								// if there is no track, clear the flag
	}
EUREKA!!!! THERE IT IS!!!! AND IT'S NOT COMMENTED OUT!

Thanks Mike! Your post led me to find the real non-commented out camera handler!!!

It appears the answer is NO, important parts of the camera code ARE NOT commented out. This SHOULD still work out of the box with little to no code modification

-SlimBoJones...

P.S. My question has been answered! Moderators may close this thread if they wish.

Last edited by Mr. Lim : 14-01-2005 at 12:15. Reason: Added thread closure request
 


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

Similar Threads
Thread Thread Starter Forum Replies Last Post
[moderated] Spare Parts Rule for 2005 Ken Patton General Forum 20 11-06-2005 14:19
Sourceforge for Code Repository and other stuff SilverStar Programming 9 15-01-2005 21:16
Coding / Style Standards for sharing C code Joe Johnson Programming 33 01-05-2004 15:15
PLEASE RESPOND: NRLB & Spare Parts at Nats ngreen Championship Event 12 23-04-2004 16:06
Rules on making spare parts fyi DougHogg General Forum 0 02-04-2003 16:18


All times are GMT -5. The time now is 02:37.

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