{"action":"closed","issue":{"url":"https://api.github.com/repos/NESCent/popgenInfo/issues/163","repository_url":"https://api.github.com/repos/NESCent/popgenInfo","labels_url":"https://api.github.com/repos/NESCent/popgenInfo/issues/163/labels{/name}","comments_url":"https://api.github.com/repos/NESCent/popgenInfo/issues/163/comments","events_url":"https://api.github.com/repos/NESCent/popgenInfo/issues/163/events","html_url":"https://github.com/NESCent/popgenInfo/issues/163","id":139370795,"number":163,"title":"Issues to be fixed before publication","user":{"login":"zkamvar","id":3639446,"avatar_url":"https://avatars.githubusercontent.com/u/3639446?v=3","gravatar_id":"","url":"https://api.github.com/users/zkamvar","html_url":"https://github.com/zkamvar","followers_url":"https://api.github.com/users/zkamvar/followers","following_url":"https://api.github.com/users/zkamvar/following{/other_user}","gists_url":"https://api.github.com/users/zkamvar/gists{/gist_id}","starred_url":"https://api.github.com/users/zkamvar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zkamvar/subscriptions","organizations_url":"https://api.github.com/users/zkamvar/orgs","repos_url":"https://api.github.com/users/zkamvar/repos","events_url":"https://api.github.com/users/zkamvar/events{/privacy}","received_events_url":"https://api.github.com/users/zkamvar/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"zkamvar","id":3639446,"avatar_url":"https://avatars.githubusercontent.com/u/3639446?v=3","gravatar_id":"","url":"https://api.github.com/users/zkamvar","html_url":"https://github.com/zkamvar","followers_url":"https://api.github.com/users/zkamvar/followers","following_url":"https://api.github.com/users/zkamvar/following{/other_user}","gists_url":"https://api.github.com/users/zkamvar/gists{/gist_id}","starred_url":"https://api.github.com/users/zkamvar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zkamvar/subscriptions","organizations_url":"https://api.github.com/users/zkamvar/orgs","repos_url":"https://api.github.com/users/zkamvar/repos","events_url":"https://api.github.com/users/zkamvar/events{/privacy}","received_events_url":"https://api.github.com/users/zkamvar/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/NESCent/popgenInfo/milestones/1","html_url":"https://github.com/NESCent/popgenInfo/milestones/Publication","labels_url":"https://api.github.com/repos/NESCent/popgenInfo/milestones/1/labels","id":1107239,"number":1,"title":"Publication","description":"","creator":{"login":"zkamvar","id":3639446,"avatar_url":"https://avatars.githubusercontent.com/u/3639446?v=3","gravatar_id":"","url":"https://api.github.com/users/zkamvar","html_url":"https://github.com/zkamvar","followers_url":"https://api.github.com/users/zkamvar/followers","following_url":"https://api.github.com/users/zkamvar/following{/other_user}","gists_url":"https://api.github.com/users/zkamvar/gists{/gist_id}","starred_url":"https://api.github.com/users/zkamvar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zkamvar/subscriptions","organizations_url":"https://api.github.com/users/zkamvar/orgs","repos_url":"https://api.github.com/users/zkamvar/repos","events_url":"https://api.github.com/users/zkamvar/events{/privacy}","received_events_url":"https://api.github.com/users/zkamvar/received_events","type":"User","site_admin":false},"open_issues":8,"closed_issues":7,"state":"open","created_at":"2015-05-12T17:43:34Z","updated_at":"2016-03-08T22:28:51Z","due_on":"2016-03-16T04:00:00Z","closed_at":null},"comments":2,"created_at":"2016-03-08T19:27:18Z","updated_at":"2016-03-08T22:28:51Z","closed_at":"2016-03-08T22:28:51Z","body":"Before publication, all vignettes and content should be ship shape. I will list issues here, no matter how small. **critical issues are bold**\r\n\r\n## Vignettes\r\n\r\n- [ ] [Microsatellite basic statistics issues](http://nescent.github.io/popgenInfo/startMicrosatellite.html):\r\n  - [x] **What's next link is broken**\r\n  - [x] add message = FALSE to suppress package startup messages in package loading chunk\r\n  - [x] Fis and Fst are mentioned, but not tested (Perhaps it can be copied from the SNP vignette)\r\n  - [x] Assumptions should be list\r\n  - [x] Some grammar/style needs to be fixed\r\n  - [x] The object `Mydata` is not necessary as `hw.test()` works on genind objects.\r\n- [ ] [Start SNP](http://nescent.github.io/popgenInfo/StartSNP.html)\r\n  - [x] Link to data set should span text beyond the word \"here\".\r\n  - [x] grammar/style\r\n- [ ] [Differentiation SNP](http://nescent.github.io/popgenInfo/DifferentiationSNP.html)\r\n  - [x] **What's next link is broken**\r\n  - [x] Package declarations should be quoted\r\n  - [x] Link to data set should span text beyond the word \"here\".\r\n  - [x] **`set.seed()` needs to be called before `find.clusters()`** If it's not, the cluster assignment could be 5 or 6 clusters, making the statement in the text inaccurate.\r\n  - [ ] The statement \"Five groups were finally detected and represented by the discriminant analysis. The 11 states are not the good level for population analysis. But we need to remember that we used only 20 SNPs to conduct the analysis.\" doesn't make much sense to me. Why would we say that the 11 stats are not good for population analysis?\r\n  - [ ] grammar/style\r\n- [ ] [SNP distances](http://nescent.github.io/popgenInfo/2015-05-18-Dist-SNP.html)\r\n  - [ ] Link to data set should span text beyond the word \"here\".\r\n  - [ ] `distgen()` in text should read `dist.gene()`\r\n  - [ ] remove mention of gstudio since the package is not being updated on CRAN\r\n- [ ] [Summary Stats for Sequences](SummarySequenceData.html) needs to be added or removed from menu and manuscript\r\n- [ ] [Population Differentiation Sequence](http://nescent.github.io/popgenInfo/PopDiffSequenceData.html)\r\n  - [ ] Data import section could utilize `apex::read.multiFASTA()`\r\n  - [ ]  Need reference for AMOVA \". Furthermore, it is possible to test hypotheses about differentiation by grouping subpopulations in a hierarchical structure (Ref).\"\r\n\r\n## Package Developement\r\n\r\n- [ ] [Data Formats](http://nescent.github.io/popgenInfo/DATAFORMATS.html) needs multiDNA and multiPhyDat added (@thibautjombart and @KlausVigo)\r\n- [ ] **[Packages Available in R](http://nescent.github.io/popgenInfo/PACKAGES.html) needs to be updated.** There are many sentence fragments for some reason (See #122)\r\n- [ ]  [Introduction](http://nescent.github.io/popgenInfo/DEVELOPER_INTRO.html) and [Developing R Packages](http://nescent.github.io/popgenInfo/PACKAGE_TUTORIALS.html) are fairly redundant. Probably should merge former into latter.\r\n\r\n## Other\r\n\r\n- [ ] in [Contribution guidelines](CONTRIBUTING.md), remove \"submit early\" from best practices\r\n- [ ] [Getting Started](http://nescent.github.io/popgenInfo/GETSTARTED.html) is pretty bare. It needs at least a description of what those links are for."}}