Skip to content

Make transform definitions own editable tuning defaults - #159

Merged
KeyffMS merged 5 commits into
mainfrom
agent/issue-114-transform-tuning-authority-v2
Sep 24, 2026
Merged

KeyffMS merged 5 commits into
mainfrom
agent/issue-114-transform-tuning-authority-v2

Conversation

@KeyffMS

@KeyffMS KeyffMS commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Addresses #114.

Changes:

  • makes each VisualProfileDefinition own canonical/reset tuning as well as normalization;
  • resolves VisualProfileManagementService.UpdateTuning through the target profile's registered transform definition;
  • removes the editor's hard-coded Soft Invert reset label and reset values;
  • adds a test-capable catalog constructor without introducing another registry or plugin framework;
  • adds a test-only second editable transform proving that update normalization and editor reset use its own definition;
  • preserves the existing Soft Invert defaults, numeric clamping and persisted values;
  • documents VisualProfileDefinition as the tuning authority.

…talog.cs

Addresses #114.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
…nagementService.cs

Addresses #114.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
…itorForm.cs

Addresses #114.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
…rofileTuningAuthorityTests.cs

Addresses #114.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
Addresses #114.

Signed-off-by: KeyffMS <124252104+KeyffMS@users.noreply.github.com>
@KeyffMS
KeyffMS merged commit f7ea72f into main Sep 24, 2026
2 checks passed
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