Skip to content

Use model metadata name in browser#658

Draft
ArdaUsta13 wants to merge 1 commit into
sdatkinson:mainfrom
ArdaUsta13:use-name-to-populate-modelname
Draft

Use model metadata name in browser#658
ArdaUsta13 wants to merge 1 commit into
sdatkinson:mainfrom
ArdaUsta13:use-name-to-populate-modelname

Conversation

@ArdaUsta13

Copy link
Copy Markdown

Summary

  • Use .nam metadata.name as the model display label when present.
  • Preserve the file path in the UI/delegate payload so the selected model still loads correctly.
  • Show metadata names in the model dropdown/menu, with filename fallback for models without a metadata name.

Fixes #264.

Validation

  • git diff --check -- NeuralAmpModeler/NeuralAmpModeler.cpp NeuralAmpModeler/NeuralAmpModeler.h NeuralAmpModeler/NeuralAmpModelerControls.h
  • xcodebuild -project NeuralAmpModeler/projects/NeuralAmpModeler-macOS.xcodeproj -scheme macOS-APP -configuration Debug -derivedDataPath /private/tmp/nam-xcodebuild-derived APP_PATH=/private/tmp/nam-apps CODE_SIGNING_ALLOWED=NO build
  • xcodebuild -project NeuralAmpModeler/projects/NeuralAmpModeler-macOS.xcodeproj -scheme macOS-VST3 -configuration Debug -derivedDataPath /private/tmp/nam-xcodebuild-derived VST3_PATH=/private/tmp/nam-vst3 CODE_SIGNING_ALLOWED=NO build

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.

[FEATURE] Use "name" in metadata to populate model name string

1 participant