Skip to content

feat: make optimal-P estimation default (no --estimate-optimal-p flag required) #147

@nerdCopter

Description

@nerdCopter

Context

Optimal P estimation is currently opt-in via --estimate-optimal-p. The flag was added conservatively while the feature was experimental, but the analysis requires only setpoint[3] (throttle channel) which is present in nearly all Betaflight/EmuFlight logs.

Proposal

Gate the feature on setpoint[3] availability rather than an explicit flag. If throttle data is present, run the torque-inertia profiler and optimal-P analysis automatically alongside the existing step-response output.

Considerations

  • The --estimate-optimal-p flag would be removed (or silently accepted and ignored for backwards compatibility).
  • This work is naturally coupled with the double-parse fix — see perf/feat: eliminate double-parse and make optimal-P estimation default (no flag required) #146 — since removing the flag means profiling always runs, making the unified single-parse path even more important.
  • The feature label Optimal P (Experimental, log-derived) already signals experimental status to users.

References

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