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

Add the option to ignore case #96

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

steveharoz
Copy link

Statcheck ignores statistical tests if the letter case is wrong such as the incorrect "f(12,34) = 0.56, P = 0.048" instead of correct "F(12,34) = 0.56, p = 0.048".

While I understand the value in checking for proper APA formatting, confusing upper and lower case usage is reasonably common, especially for F-tests. I suspect that the more common goal of a StatCheck user is to find and check as many reported statistics as possible, rather than only those the perfectly comply with APA formatting. So I suggest optimizing for that use case by defaulting to ignoring case and giving users the option to more strictly follow APA formatting.

Questions:

  • For the parameter ignore_case, would you prefer snake_case or camelCase?
  • Are there any examples where the case DOES matter? If so, it can be selectively turned on/off per regex pattern.

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.

1 participant