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.

Sign up for GitHub

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

Jump to bottom

Stream line parameter descriptions and names. #12

Open
stefanedwards opened this issue Oct 6, 2016 · 1 comment
Open

Stream line parameter descriptions and names. #12

stefanedwards opened this issue Oct 6, 2016 · 1 comment
Assignees
@stefanedwards
Labels
To do
Milestone
1.0.0 Beta

Comments

@stefanedwards
Copy link
Owner

stefanedwards commented Oct 6, 2016 •
edited
Loading

ncol, nSNPs, nAnimals, naval, NAval, numeric.format, etc. are just some of the different names for the same concepts. These need to be stream lined.

Task also requires removing nSNPs and ncol as setting this to anything other than the actual number of columns will lead to weird results.
@stefanedwards stefanedwards added this to the 1.0.0 Beta milestone Oct 6, 2016
@stefanedwards stefanedwards self-assigned this Oct 6, 2016
@stefanedwards
Copy link
Owner Author

Try to include the common arguments in a .Rd file, using the @inheritParams command in roxygen. Add also a description of integer/real format descriptions to Siccuracy 'front-page'.

stefanedwards added a commit that referenced this issue Oct 20, 2016
@stefanedwards
Updated descriptions cf. issue #12.
477cb67 
`ncols` are kept to allow user some efficiency.
convert_phase is reduced to a single subroutine that checks either way.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stefanedwards stefanedwards

Labels
To do
Projects
None yet
Milestone
1.0.0 Beta
Development

No branches or pull requests
1 participant
@stefanedwards