Skip to content

Improvement to pspm_text#756

Merged
dominikbach merged 3 commits intodevelopfrom
help-text
Aug 12, 2024
Merged

Improvement to pspm_text#756
dominikbach merged 3 commits intodevelopfrom
help-text

Conversation

@dominikbach
Copy link
Copy Markdown
Contributor

@dominikbach dominikbach commented Aug 8, 2024

Fixes #657 .

Should be reviewed AFTER #735 is merged.

Changes proposed in this pull request:

  • integrates all import help texts contained in pspm_text into the respective pspm_get_... functions
  • formats these help texts in the matlabbatch
  • prunes unused warning texts from pspm_text
  • renames pspm_text into pspm_help_text for future use with help items
  • converts pspm_help_text into a function so that saving of the file pspm_text.mat is no longer necessary

@dominikbach dominikbach requested a review from teddphil August 8, 2024 10:14
@dominikbach dominikbach self-assigned this Aug 8, 2024
@dominikbach dominikbach added Completed & Waiting for Review Completed and waiting for review Improvement A minor improvement to the project labels Aug 8, 2024
@dominikbach dominikbach added this to the v7.0 milestone Aug 8, 2024
@dominikbach dominikbach mentioned this pull request Aug 9, 2024
Comment thread src/pspm_init.m Outdated
pspm_text([pspm_root, filesep]);
load(fullfile(pspm_root,'pspm_text.mat'))

% 2.5 Add required paths ---
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There is no 2.4 and two 2.7

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed.

Comment thread src/pspm_trim.m
numel(g_endmarker) ~= 1 || ...
startmarker < 1 || ...
g_endmarker < startmarker
warning('ID:invalid_input', 'No valid reference markers.\n'); return
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Think at line 237, the ndata can be changed to data_struct if preferred...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

changed

Copy link
Copy Markdown

@teddphil teddphil left a comment

Choose a reason for hiding this comment

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

Only minor comments...

@teddphil teddphil added Approved The pull request has been approved and can be checked and then merged. and removed Completed & Waiting for Review Completed and waiting for review labels Aug 12, 2024
@dominikbach dominikbach merged commit 3a7ec21 into develop Aug 12, 2024
@dominikbach dominikbach deleted the help-text branch August 12, 2024 08:02
@teddphil teddphil removed Improvement A minor improvement to the project Approved The pull request has been approved and can be checked and then merged. labels Sep 9, 2024
@teddphil teddphil mentioned this pull request Oct 12, 2024
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.

Extend or remove pspm_text.m

2 participants