You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the PID class provided in the header file lacks public methods to access the current value of error_sum and to reset it to zero. These functionalities are essential for users who want to monitor the error sum or reset it as needed during runtime. For example, the getter would enable users of the library to know when a wheel has locked up or power has been lost, and the reset method would provide the ability to remediate that error state.
Proposed Solution:
To address this limitation, I propose adding two public methods to the PID class:
float getErrorSum() const;: This method will allow users to retrieve the current value of error_sum without modifying it.
void resetErrorSum();: This method will reset the value of error_sum to zero, enabling users to clear accumulated error terms when necessary.
Expected Benefits:
Enhanced usability: Users will have convenient access to monitor and manage the error sum within the PID controller.
Improved flexibility: The addition of these methods will make the PID class more versatile and adaptable to different control scenarios.
Impact Assessment:
This enhancement should have minimal impact on existing codebases as it only adds two simple public methods to the PID class. However, it greatly enhances the usability and flexibility of the class.
Additional Notes:
Consideration should be given to documenting these new methods appropriately in the class documentation to ensure users understand their purpose and usage.
Thank you for considering this improvement to the PID class. Please let me know if further clarification or details are needed.
The text was updated successfully, but these errors were encountered:
Currently, the PID class provided in the header file lacks public methods to access the current value of error_sum and to reset it to zero. These functionalities are essential for users who want to monitor the error sum or reset it as needed during runtime. For example, the getter would enable users of the library to know when a wheel has locked up or power has been lost, and the reset method would provide the ability to remediate that error state.
Proposed Solution:
To address this limitation, I propose adding two public methods to the PID class:
Expected Benefits:
Impact Assessment:
This enhancement should have minimal impact on existing codebases as it only adds two simple public methods to the PID class. However, it greatly enhances the usability and flexibility of the class.
Additional Notes:
Consideration should be given to documenting these new methods appropriately in the class documentation to ensure users understand their purpose and usage.
Thank you for considering this improvement to the PID class. Please let me know if further clarification or details are needed.
The text was updated successfully, but these errors were encountered: