Skip to content

Route standalone structeditor to modern app#82

Draft
ehennestad wants to merge 4 commits intodevfrom
codex/modern-structeditor-integration
Draft

Route standalone structeditor to modern app#82
ehennestad wants to merge 4 commits intodevfrom
codex/modern-structeditor-integration

Conversation

@ehennestad
Copy link
Copy Markdown
Collaborator

Summary

  • Route R2025a+ standalone structeditor calls to the modern standalone StructEditorApp while keeping legacy for old MATLAB, embedded/parent-handle, and forced legacy paths.
  • Add the NANSEN OptionsManagerPlugin for preset selection, save/default controls, and modified-option tracking in modern standalone editors.
  • Migrate standalone callers to the modern Data/OriginalData/FinishState contract and add a disposable OptionsManager plugin smoke fixture.

Notes

  • Requires the companion standalone StructEditor branch codex/nansen-compatibility-contract.
  • Roimanager embedding remains on the legacy path and is intentionally out of scope.

@ehennestad ehennestad added this to the R2025+ Support milestone May 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Test Results

185 tests  +5   180 ✅ ±0   25s ⏱️ +3s
 10 suites +1     5 💤 +5 
  1 files   ±0     0 ❌ ±0 

Results for commit 0e97c90. ± Comparison against base commit 16ef011.

♻️ This comment has been updated with latest results.

@ehennestad ehennestad force-pushed the codex/modern-structeditor-integration branch from c026cfb to 38b7a77 Compare May 5, 2026 17:59
obj.ImageStack.MetaData.SampleRate = editedData.SampleRate;
obj.ImageStack.MetaData.SpatialPosition = editedData.SpatialPosition;
physSize = editedData.PhysicalSize;
if numel(editedData.PhysicalUnits) == 1
@ehennestad ehennestad force-pushed the codex/modern-structeditor-integration branch from 8ebb318 to fde2b11 Compare May 6, 2026 06:06
@ehennestad ehennestad force-pushed the codex/modern-structeditor-integration branch from 52d61e3 to f390828 Compare May 6, 2026 14:07
@ehennestad ehennestad marked this pull request as draft May 7, 2026 11:02
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.

2 participants