Skip to content

Conversation

@ehennestad
Copy link
Collaborator

Make sure the list of path names for calculator document definitions is a cell array or chars, not a cell array of strings.

This fixes a bug which would cause the following error if multiple calculator repositories are on MATLAB's search path:

    ---------
    Error ID:
    ---------
    'MATLAB:dir:invalidInputType'
    --------------
    Error Details:
    --------------
    Error using dir
    Name must be a text scalar.
    
    Error in did.document.readjsonfilelocation (line 783)
                        files = dir([mypaths{j} filesep '**']);
    
    Error in ndi.document.readblankdefinition (line 845)
                t = did.document.readjsonfilelocation(jsonfilelocationstring);
    
    Error in ndi.document (line 28)
                    document_properties = ndi.document.readblankdefinition(document_type);
    
    Error in ndi.session.dir (line 98)
                    g = ndi.document('session','session.reference',ndi_session_dir_obj.reference) + ...

@ehennestad ehennestad requested a review from stevevanhooser May 5, 2025 09:37
Copy link
Contributor

@stevevanhooser stevevanhooser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@stevevanhooser stevevanhooser merged commit 1bc439a into main May 5, 2025
@stevevanhooser stevevanhooser deleted the fix-ndi-pathconstants branch May 5, 2025 19:09
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