Skip to content

Question: Breaking changes #17

Closed
Closed

Description

@halmos

Hi, The new features look really great.

One question, after updating to the latest release I've found that my pre-existing tuning parameters no longer produce the same output.

To update to the latest class constructor parameters, I set the DOn value to 0. Is that the value you would recommend to emulate the QuickPid behavior prior to the addition of the DOn feature?

Or, do you have any idea about what might be the cause of the altered PID behavior or how I might compensate without having to re-tune my PID values?

Many thanks.

Activity

  1. Dlloydev commented on Jul 21, 2021

    @Dlloydev
    Owner

    The probable cause would be the addition of integral anti-windup. You could test this by commenting out lines 69-71 in QuickPID.cpp. Otherwise, you may need to increase the Ki value or re-tune.

    I have the default POn value at 1, DOn value at 0 which is similar to the default behavior of Arduino PID.

  2. halmos commented on Jul 27, 2021

    @halmos
    Author

    Thanks - that's helpful. It does seem to be the windup.

  3. Repository owner locked and limited conversation to collaborators on Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions