{"action":"created","issue":{"url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/24","labels_url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/24/comments","events_url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/24/events","html_url":"https://github.com/ismrmrd/ismrmrd/pull/24","id":70449829,"number":24,"title":"For upstream","user":{"login":"mklassen","id":1391926,"avatar_url":"https://avatars.githubusercontent.com/u/1391926?v=3","gravatar_id":"","url":"https://api.github.com/users/mklassen","html_url":"https://github.com/mklassen","followers_url":"https://api.github.com/users/mklassen/followers","following_url":"https://api.github.com/users/mklassen/following{/other_user}","gists_url":"https://api.github.com/users/mklassen/gists{/gist_id}","starred_url":"https://api.github.com/users/mklassen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mklassen/subscriptions","organizations_url":"https://api.github.com/users/mklassen/orgs","repos_url":"https://api.github.com/users/mklassen/repos","events_url":"https://api.github.com/users/mklassen/events{/privacy}","received_events_url":"https://api.github.com/users/mklassen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":22,"created_at":"2015-04-23T16:09:40Z","updated_at":"2015-04-24T18:12:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ismrmrd/ismrmrd/pulls/24","html_url":"https://github.com/ismrmrd/ismrmrd/pull/24","diff_url":"https://github.com/ismrmrd/ismrmrd/pull/24.diff","patch_url":"https://github.com/ismrmrd/ismrmrd/pull/24.patch"},"body":"Added ```const char *``` accessors for ```Image<T>``` attribute string to simplify copying to/from ```char *``` by avoiding creation of intermediate ```std::string``` copy.\r\n\r\nAdded missing Dataset::readImage instantiations and Boost include for tests."},"comment":{"url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/comments/96019353","html_url":"https://github.com/ismrmrd/ismrmrd/pull/24#issuecomment-96019353","issue_url":"https://api.github.com/repos/ismrmrd/ismrmrd/issues/24","id":96019353,"user":{"login":"inati","id":3442024,"avatar_url":"https://avatars.githubusercontent.com/u/3442024?v=3","gravatar_id":"","url":"https://api.github.com/users/inati","html_url":"https://github.com/inati","followers_url":"https://api.github.com/users/inati/followers","following_url":"https://api.github.com/users/inati/following{/other_user}","gists_url":"https://api.github.com/users/inati/gists{/gist_id}","starred_url":"https://api.github.com/users/inati/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/inati/subscriptions","organizations_url":"https://api.github.com/users/inati/orgs","repos_url":"https://api.github.com/users/inati/repos","events_url":"https://api.github.com/users/inati/events{/privacy}","received_events_url":"https://api.github.com/users/inati/received_events","type":"User","site_admin":false},"created_at":"2015-04-24T18:12:58Z","updated_at":"2015-04-24T18:12:58Z","body":"I knew this was going to be a mess.  I think @naegelejd is correct that the length in the header should not include the null and that the null is just a c implementation detail so that we can treat it as a c-string and that it should not be in the hdf5 or on the wire. But I want to hear what @hansenms has to say before I fully commit. "}}