Skip to content

qtgui: expose Reset for Geometry numeric controls - #389

Draft
janhubicka wants to merge 1 commit into
mainfrom
agent/qtgui-geometry-default-reset
Draft

janhubicka wants to merge 1 commit into
mainfrom
agent/qtgui-geometry-default-reset

Conversation

@janhubicka

Copy link
Copy Markdown
Owner

Summary

Finish the small safe Geometry portion of the Phase C default/modified-state rollout.

  • Opt these numeric saved controls into real-default / modified / Reset presentation:
    • Lens center distance
    • Final rotation
  • Preserve 0 = Auto for Lens center distance; it is also the real ParameterState default.
  • Reset remains a separate undoable gesture.
  • Leave scanner-geometry enums, fit-request/session toggles, and fitted lens/tilt results on their existing dedicated semantics.
  • Extend workspace churn to require default/reset metadata for both migrated Geometry keys.

No geometry solver behavior, fitted calibration result, .par format, or parameter default changes.

This PR is independent of the other Phase C PRs rather than stacked on them.

[ci:ubuntu-checking]

Finish the small safe Geometry portion of the Phase C default/modified-state
rollout.

Opt Lens center distance and Final rotation into ParameterPanel's real-default
presentation. Lens center distance retains its explicit 0 = Auto special value,
which is also the real ParameterState default; Final rotation resets to the
fresh-state orientation default. Both remain separate undoable reset gestures.

Do not add Reset to scanner geometry enums, fit-request toggles, nonlinear
session state, or fitted lens/tilt results; those are discrete/calibration
semantics already handled by their dedicated controls.

Extend workspace churn to require default/reset metadata for both migrated
Geometry keys.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant