{"action":"created","issue":{"url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8","labels_url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8/comments","events_url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8/events","html_url":"https://github.com/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8","id":101069941,"number":8,"title":"Error if vcf doesn't start with sample info","user":{"login":"kveeramah","id":13582227,"avatar_url":"https://avatars.githubusercontent.com/u/13582227?v=3","gravatar_id":"","url":"https://api.github.com/users/kveeramah","html_url":"https://github.com/kveeramah","followers_url":"https://api.github.com/users/kveeramah/followers","following_url":"https://api.github.com/users/kveeramah/following{/other_user}","gists_url":"https://api.github.com/users/kveeramah/gists{/gist_id}","starred_url":"https://api.github.com/users/kveeramah/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kveeramah/subscriptions","organizations_url":"https://api.github.com/users/kveeramah/orgs","repos_url":"https://api.github.com/users/kveeramah/repos","events_url":"https://api.github.com/users/kveeramah/events{/privacy}","received_events_url":"https://api.github.com/users/kveeramah/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-14T18:13:27Z","updated_at":"2015-08-14T20:01:09Z","closed_at":null,"body":"Just noticed that if you run a file where the first line is not the sample info, then lcMKkin will crash. This is quite an important change as many vcf creators will produce files with these leading information lines.\r\n\r\nMany vcf creators will have a  bunch of other stuff at the beginning of the vcf, indicated by a leading double hash. The sample info is indicated by only a single hash. \r\n\r\nFor example a vcf file might be like so:\r\n##GATKstuff\r\n##More GATKstuff\r\n#CHROM  POS     ID      REF     ALT     QUAL    FILTER  INFO    FORMAT  SAMPLE1 SAMPLE2\r\n\r\nWe want to ignore those double hashes and only start reading the single hash as the first line with the sample name info."},"comment":{"url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/comments/131220689","html_url":"https://github.com/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8#issuecomment-131220689","issue_url":"https://api.github.com/repos/COMBINE-lab/maximum-likelihood-relatedness-estimation/issues/8","id":131220689,"user":{"login":"rob-p","id":361470,"avatar_url":"https://avatars.githubusercontent.com/u/361470?v=3","gravatar_id":"","url":"https://api.github.com/users/rob-p","html_url":"https://github.com/rob-p","followers_url":"https://api.github.com/users/rob-p/followers","following_url":"https://api.github.com/users/rob-p/following{/other_user}","gists_url":"https://api.github.com/users/rob-p/gists{/gist_id}","starred_url":"https://api.github.com/users/rob-p/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rob-p/subscriptions","organizations_url":"https://api.github.com/users/rob-p/orgs","repos_url":"https://api.github.com/users/rob-p/repos","events_url":"https://api.github.com/users/rob-p/events{/privacy}","received_events_url":"https://api.github.com/users/rob-p/received_events","type":"User","site_admin":false},"created_at":"2015-08-14T20:01:09Z","updated_at":"2015-08-14T20:01:09Z","body":"@kveeramah; do you have a file that you can test this on to confirm that my fix addresses this crash.  The \"correct\" eventual solution is to rely on a proper VCF parsing library (I've started integrating [vcflib](https://github.com/ekg/vcflib)), but this should fix this problem for the time being."}}