{"action":"created","issue":{"url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/72","labels_url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/72/labels{/name}","comments_url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/72/comments","events_url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/72/events","html_url":"https://github.com/ComputationalRadiationPhysics/haseongpu/pull/72","id":91104355,"number":72,"title":"parsing completely remastered + config file","user":{"login":"slizzered","id":903310,"avatar_url":"https://avatars.githubusercontent.com/u/903310?v=3","gravatar_id":"","url":"https://api.github.com/users/slizzered","html_url":"https://github.com/slizzered","followers_url":"https://api.github.com/users/slizzered/followers","following_url":"https://api.github.com/users/slizzered/following{/other_user}","gists_url":"https://api.github.com/users/slizzered/gists{/gist_id}","starred_url":"https://api.github.com/users/slizzered/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slizzered/subscriptions","organizations_url":"https://api.github.com/users/slizzered/orgs","repos_url":"https://api.github.com/users/slizzered/repos","events_url":"https://api.github.com/users/slizzered/events{/privacy}","received_events_url":"https://api.github.com/users/slizzered/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/labels/cleanup","name":"cleanup","color":"d4c5f9"},{"url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/labels/new%20feature","name":"new feature","color":"0052cc"}],"state":"open","locked":false,"assignee":{"login":"erikzenker","id":341041,"avatar_url":"https://avatars.githubusercontent.com/u/341041?v=3","gravatar_id":"","url":"https://api.github.com/users/erikzenker","html_url":"https://github.com/erikzenker","followers_url":"https://api.github.com/users/erikzenker/followers","following_url":"https://api.github.com/users/erikzenker/following{/other_user}","gists_url":"https://api.github.com/users/erikzenker/gists{/gist_id}","starred_url":"https://api.github.com/users/erikzenker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/erikzenker/subscriptions","organizations_url":"https://api.github.com/users/erikzenker/orgs","repos_url":"https://api.github.com/users/erikzenker/repos","events_url":"https://api.github.com/users/erikzenker/events{/privacy}","received_events_url":"https://api.github.com/users/erikzenker/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2015-06-25T23:36:30Z","updated_at":"2015-06-30T23:10:02Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/pulls/72","html_url":"https://github.com/ComputationalRadiationPhysics/haseongpu/pull/72","diff_url":"https://github.com/ComputationalRadiationPhysics/haseongpu/pull/72.diff","patch_url":"https://github.com/ComputationalRadiationPhysics/haseongpu/pull/72.patch"},"body":" Maybe there is some stuff missing. Code seems to run great, but it would be good to play with the command-line parameters a bit before accepting the PR.\r\n\r\n### Things that are included:\r\n - there is now a --config parameter which can be used to supply a\r\n   config-file. The c-example already contains an example file.\r\n   The sections ([Compute] etc.) should be commented out!\r\n - --help and --config can not be defined in the config-file... this is\r\n   intentional.\r\n - all command-line parameters are no longer directly written in code,\r\n   but abstracted away into the structs in `src/types.cc`\r\n - the command-line parameters are now structured similarly to the\r\n   existing structs `ComputeParameters` and `ExperimentParameters`\r\n - The functions `fileToVector` and `fileToValue` now return the result\r\n   instead of taking a reference.\r\n - the `parse` function was completely remastered and is much smaller\r\n   now. It uses the `variables_map` to encapsulate all command-line\r\n   parameters. I used my own type for the `variables_map`, so we can\r\n   still modify the variables after parsing.\r\n - DeviceMode and ParallelMode are no longer enums, but std::string.\r\n   This is not ideal, but it makes the code much nicer.\r\n - As a result, all the nice stuff from #69 was removed again (no longer necessary)\r\n - closes #10 "},"comment":{"url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/comments/117369600","html_url":"https://github.com/ComputationalRadiationPhysics/haseongpu/pull/72#issuecomment-117369600","issue_url":"https://api.github.com/repos/ComputationalRadiationPhysics/haseongpu/issues/72","id":117369600,"user":{"login":"slizzered","id":903310,"avatar_url":"https://avatars.githubusercontent.com/u/903310?v=3","gravatar_id":"","url":"https://api.github.com/users/slizzered","html_url":"https://github.com/slizzered","followers_url":"https://api.github.com/users/slizzered/followers","following_url":"https://api.github.com/users/slizzered/following{/other_user}","gists_url":"https://api.github.com/users/slizzered/gists{/gist_id}","starred_url":"https://api.github.com/users/slizzered/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/slizzered/subscriptions","organizations_url":"https://api.github.com/users/slizzered/orgs","repos_url":"https://api.github.com/users/slizzered/repos","events_url":"https://api.github.com/users/slizzered/events{/privacy}","received_events_url":"https://api.github.com/users/slizzered/received_events","type":"User","site_admin":false},"created_at":"2015-06-30T23:10:02Z","updated_at":"2015-06-30T23:10:02Z","body":"... for some reason, the CI :white_check_mark: does not show up for me... But the CI-build was actually done successfully. If I log into `drone.io` directly, results are fine. :grey_question: "}}