Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

If a stimConfig file is loaded at the CLI, this is not reported in the GUI #123

Open
raacampbell opened this issue Mar 15, 2023 · 0 comments
Assignees
Labels
bug Something isn't working Low priority

Comments

@raacampbell
Copy link
Contributor

Loading a stim config file at the CLI does not update the GUI state to indicate this happened.

To Reproduce
Steps to reproduce the behavior:

  1. start_zapit
  2. fn = fullfile(zapit.updater.getInstallPath,'examples','example_stimulus_config_files','uniAndBilateral_5_conditions.yml');
  3. hZP.loadStimConfig(fn)
  4. hZP.stimConfig % Yes it is loaded

But GUI does not report this.

How to fix

The code that loads the file and updates the GUI in zapit.gui.main.controller.loadStimConfig_Callback needs to be separated. The GUI update should run based on a callback that fires when the stimConfig changes.

@raacampbell raacampbell added the bug Something isn't working label Mar 15, 2023
@raacampbell raacampbell self-assigned this Mar 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Low priority
Projects
None yet
Development

No branches or pull requests

1 participant