Skip to content

setByTrigger: make sure that defaults for this can actually be set #106

Description

@Remi-Gau

See summary of the problem below

function [cfg] = checkParameters(cfg)
    %

    ...

    cfg = setDefaultFields(cfg, fieldsToSet);

    cfg = setParametersMtMst(cfg);

    cfg = setMRI(cfg);

    ...

end


function cfg = setParametersMtMst(cfg)

    switch lower(cfg.design.localizer)

        case 'mt_mst'

        ...

        case 'mt'

            ...

            % SETS the value if not defined before
            fieldsToSet.timing.IBI = 4;

            ...

    end

    cfg = setDefaultFields(cfg, fieldsToSet);

end

function cfg = setMRI(cfg)

    ...

    cfg = setDefaultFields(cfg, fieldsToSet);

    cfg = setPacedByTrigger(cfg);

end

function cfg = setPacedByTrigger(cfg)

    if cfg.pacedByTriggers.do

        ...
        % CANNOT override the value because it was set before in setParametersMtMst
        fieldsToSet.timing.IBI = 0;

        ...

        cfg = setDefaultFields(cfg, fieldsToSet);

    end

end

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions