search

Team-4536 / Command-Based-2015

0 stars 0 forks source link

getCycleTime method in Utilities #10

Closed stepansubbotin closed 9 years ago

stepansubbotin commented 9 years ago

On line 137 in Utilities.java you don't have to do Utilities.getTime(); The getTime method is inside of the Utilities class.

AnonymousAnion commented 9 years ago

Oh okay!

On Fri, Jul 10, 2015 at 12:02 PM, Stepan Subbotin notifications@github.com wrote:

On line 137 in Utilities.java you don't have to do Utilities.getTime(); The getTime method is inside of the Utilities class.

— Reply to this email directly or view it on GitHub https://github.com/Team-4536/Command-Based-2015/issues/10.

AnonymousAnion commented 9 years ago

We need to figure out the timer and get the cycle time working in the accel limit. We need to revisit the accel limit

AnonymousAnion commented 9 years ago

The prevTime variable in the timer wasn't being reset in teleop init but the current time was so cycle time became negative for the first code cycle run after we disabled then reenabled the robot. All fixed now. I also renamed the methods to getCycleTime and getTime instead of Utilities.getCycleTime and Utilities.getTime when they were used inside Utilities.

AnonymousAnion commented 9 years ago

I forgot the picture!

image

inkling16 commented 9 years ago

Don't forget your semi-colons, or Hitler will come get you.

Caleb

On Thu, Jul 23, 2015 at 5:26 PM, AnonymousAnion notifications@github.com wrote:

I forgot the picture!

[image: image] https://cloud.githubusercontent.com/assets/10052290/8863626/f3bfa6b0-315f-11e5-8d41-5a65f7c06387.png

— Reply to this email directly or view it on GitHub https://github.com/Team-4536/Command-Based-2015/issues/10#issuecomment-124255001 .