{"action":"created","issue":{"url":"https://api.github.com/repos/IQuOD/AutoQC/issues/118","labels_url":"https://api.github.com/repos/IQuOD/AutoQC/issues/118/labels{/name}","comments_url":"https://api.github.com/repos/IQuOD/AutoQC/issues/118/comments","events_url":"https://api.github.com/repos/IQuOD/AutoQC/issues/118/events","html_url":"https://github.com/IQuOD/AutoQC/pull/118","id":121958069,"number":118,"title":"Tidying up output when using parallel running","user":{"login":"s-good","id":8092601,"avatar_url":"https://avatars.githubusercontent.com/u/8092601?v=3","gravatar_id":"","url":"https://api.github.com/users/s-good","html_url":"https://github.com/s-good","followers_url":"https://api.github.com/users/s-good/followers","following_url":"https://api.github.com/users/s-good/following{/other_user}","gists_url":"https://api.github.com/users/s-good/gists{/gist_id}","starred_url":"https://api.github.com/users/s-good/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/s-good/subscriptions","organizations_url":"https://api.github.com/users/s-good/orgs","repos_url":"https://api.github.com/users/s-good/repos","events_url":"https://api.github.com/users/s-good/events{/privacy}","received_events_url":"https://api.github.com/users/s-good/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-12-14T01:19:32Z","updated_at":"2015-12-15T00:14:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/IQuOD/AutoQC/pulls/118","html_url":"https://github.com/IQuOD/AutoQC/pull/118","diff_url":"https://github.com/IQuOD/AutoQC/pull/118.diff","patch_url":"https://github.com/IQuOD/AutoQC/pull/118.patch"},"body":"The output to screen was not working well when running in parallel as messages were printed to screen for each file that was processed rather than for the full results. These changes are aimed at improving this and the percent process indicator has been removed - I may try to reincorporate this in future as I found it quite useful to track progress.\r\n\r\nIn the course of doing this I noticed that only the output from the first file processed was being written to the csv file and I've tried to fix this.\r\n\r\nI have removed the logfile writing function as this is now unused and was previously found to be very slow.\r\n\r\nIt may be better in future to read the profile list in the main function like is now done for the list of QC checks so that the QC checks can see all the profiles rather than just the ones in the same file as the profile currently being QCed. This might help avoid the need for careful division of the input files for tests like the track check, although it might complicate things in other ways."},"comment":{"url":"https://api.github.com/repos/IQuOD/AutoQC/issues/comments/164601252","html_url":"https://github.com/IQuOD/AutoQC/pull/118#issuecomment-164601252","issue_url":"https://api.github.com/repos/IQuOD/AutoQC/issues/118","id":164601252,"user":{"login":"BillMills","id":1896943,"avatar_url":"https://avatars.githubusercontent.com/u/1896943?v=3","gravatar_id":"","url":"https://api.github.com/users/BillMills","html_url":"https://github.com/BillMills","followers_url":"https://api.github.com/users/BillMills/followers","following_url":"https://api.github.com/users/BillMills/following{/other_user}","gists_url":"https://api.github.com/users/BillMills/gists{/gist_id}","starred_url":"https://api.github.com/users/BillMills/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BillMills/subscriptions","organizations_url":"https://api.github.com/users/BillMills/orgs","repos_url":"https://api.github.com/users/BillMills/repos","events_url":"https://api.github.com/users/BillMills/events{/privacy}","received_events_url":"https://api.github.com/users/BillMills/received_events","type":"User","site_admin":false},"created_at":"2015-12-15T00:14:48Z","updated_at":"2015-12-15T00:14:48Z","body":"And that's why we *always* want two sets of eyes on a project - great catch on the csv recombination, I was worried that infrastructure was under-tested... Looks good, thanks for this!"}}