Major WPILib PIDController bug

I hate that every year people turn this competition into whining rather than winning. - Akash Rastogi [more]

		Chief Delphi > Technical > Programming
Major WPILib PIDController bug

Thread Tools

Rate Thread

Display Modes

17-03-2016, 20:47

virtuald

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,047

Re: Major WPILib PIDController bug

Quote:

Originally Posted by dvanvoorst

m_bufTotal should be set to 0 in setSetpoint right after doing the m_buf.clear();

I agree, that's a bug there.

The documentation should also probably specify what each of these functions should and should not do (is enable intended to clear the error? should setsetpoint always force onTarget to be false).

__________________
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

Last edited by virtuald : 17-03-2016 at 20:53.

« Previous Thread | Portal | Next Thread »

Thread Tools
Show Printable Version Email this Page
Display Modes	Rate This Thread
Switch to Linear Mode Switch to Hybrid Mode Threaded Mode	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:30.

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