prog694/joebot — comment on commit e2d0820ceb93efcb642c317330773ab635d8334a
File: src/edu/stuy/subsystems/Shooter.java line 87
By blakeelias:

Yeah I stopped calling it because JaguarSpeed doesn't know the correct tolerance. It didn't make sense for JaguarSpeed, which should represent anything that uses Jags for speed control, to know the RPM tolerance. The tolerance is something that depends on the particular use of that component, so I made it happen in Shooter instead of JaguarSpeed.

Now that I think about it, JaguarSpeed could have an argument for tolerance value and Shooter could pass it in. Then JaguarSpeed.isAtSetPoint() could check against that parameter. I'll do that now.