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

Reduce number of parameters needed to configure PA levels #1247

Closed
7 tasks done
vincentvanhees opened this issue Jan 16, 2025 · 0 comments · Fixed by #1254
Closed
7 tasks done

Reduce number of parameters needed to configure PA levels #1247

vincentvanhees opened this issue Jan 16, 2025 · 0 comments · Fixed by #1254
Assignees

Comments

@vincentvanhees
Copy link
Member

vincentvanhees commented Jan 16, 2025

There seems no value in being able to use different thresholds for MVPA between part 2 and 5. Therefore:

  • Use threshold.mod as value for mvpathreshold, if mvpathreshold is not provided.
  • Replace mvpathreshold in documentation by threshold.mod.
  • Use boutcriter.mvpa as value for boutcriter, if boutcriter is not provided.
  • Replace boutcriter in documentation by boutcriter.mvpa.
  • Correct parameter documentation for mvpadur to list c(1, 5, 10) as default because 10 is not the default.
  • Double check that other phyact parameter defaults are correctly listed.
  • Document parameter bout.metric even though it has been deprecated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant