Skip to content

Added full string support in ParamEditor#146

Merged
jkbhagatio merged 1 commit intodevfrom
ParamStrings
May 2, 2019
Merged

Added full string support in ParamEditor#146
jkbhagatio merged 1 commit intodevfrom
ParamStrings

Conversation

@k1o0
Copy link
Copy Markdown
Contributor

@k1o0 k1o0 commented Apr 29, 2019

Response to issue #134. Now an array of strings behaves the same as numerical arrays, namely...

str = [ "A" "B"
        "a" "b" ]

... is represented as two trial conditions: 'A, a' and 'B, b'.

It seems MATLAB ui tables still can't have strings as data, so the ParamEditor code converts them to char arrays then back again for the underlying parameter structure. The only current limitation is that your string parameters can't contain commas.

@kevin-j-miller
Copy link
Copy Markdown
Contributor

Looks like it's working with my expDef! Thanks for doing this, Miles!

@jkbhagatio
Copy link
Copy Markdown
Member

looks good on mc on the zrigs and passes tests

@jkbhagatio jkbhagatio merged commit 2bbd932 into dev May 2, 2019
@jkbhagatio jkbhagatio deleted the ParamStrings branch July 23, 2019 16:25
k1o0 pushed a commit that referenced this pull request Jan 3, 2021
Added full string support in ParamEditor
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.

3 participants