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

Cleanup GUI #706

Merged
merged 13 commits into from
Jun 10, 2024
Merged

Cleanup GUI #706

merged 13 commits into from
Jun 10, 2024

Conversation

dominikbach
Copy link
Contributor

@dominikbach dominikbach commented May 8, 2024

Fixes #591 .
Fixes #599 .
Fixes #669 .
Fixes #671 .
Fixes #687 .
Fixes #688 .
Fixes #715 .

Changes proposed in this pull request:

  • GUI changes according to the linked issues
  • combine data & design selector for GLM and pspm_extract_segments into a common function

@dominikbach dominikbach self-assigned this May 8, 2024
@dominikbach dominikbach added the In Progress Currently being worked on label May 8, 2024
@dominikbach dominikbach added Completed & Waiting for Review Completed and waiting for review and removed In Progress Currently being worked on labels Jun 3, 2024
@teddychao teddychao added Under Review and removed Completed & Waiting for Review Completed and waiting for review labels Jun 4, 2024
@@ -1,4 +1,4 @@
function artefact_rm = pspm_cfg_artefact_rm
Copy link
Contributor

Choose a reason for hiding this comment

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

This is fine currently. As I understand, version 7.0 will no longer include the section "Tools", so currently there is no problem. Yet if it is in 6.x versions, the pspm_guide and pspm_appdesigner, which include the module tools, will need to be updated for making them in line with these files.

tau.num = [1 1];
tau.help = {'Time constant in seconds.'};

FilterLeaky = cfg_branch;
Copy link
Contributor

Choose a reason for hiding this comment

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

I apologise if I wrote these in capitalised format like this. I have changed to use smaller case since last discussion.

%% Manual mode
mode_manual = cfg_branch;
mode_manual.name = 'Manual';
mode_manual.tag = 'mode_manual';
mode_manual.val = {datafile, channel, conditions};
mode_manual.val = {channel, timeunits, session_rep};
Copy link
Contributor

@teddychao teddychao Jun 10, 2024

Choose a reason for hiding this comment

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

Just to mention, in the new version there is no "data files" item. Instead, there is a "Data & Design". This is quite "session_rep" rather than "data file". If this is expected, please ignore me.

Extract segments

Before:
image

After:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, this is intended.

@teddychao
Copy link
Contributor

Tested these functions in the GUI and found them working ok, just a minor comment listed above.

@teddychao teddychao added Completed & Waiting for Review Completed and waiting for review and removed Under Review labels Jun 10, 2024
@dominikbach dominikbach merged commit 0f92e40 into develop Jun 10, 2024
1 check passed
@dominikbach dominikbach deleted the cleanup-GUI branch June 10, 2024 15:30
@teddychao teddychao removed the Completed & Waiting for Review Completed and waiting for review label Jun 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment