{"action":"created","issue":{"url":"https://api.github.com/repos/dondi/GRNsight/issues/314","repository_url":"https://api.github.com/repos/dondi/GRNsight","labels_url":"https://api.github.com/repos/dondi/GRNsight/issues/314/labels{/name}","comments_url":"https://api.github.com/repos/dondi/GRNsight/issues/314/comments","events_url":"https://api.github.com/repos/dondi/GRNsight/issues/314/events","html_url":"https://github.com/dondi/GRNsight/issues/314","id":169239528,"number":314,"title":"Specification of file format for GraphML export and import","user":{"login":"kdahlquist","id":6463867,"avatar_url":"https://avatars.githubusercontent.com/u/6463867?v=3","gravatar_id":"","url":"https://api.github.com/users/kdahlquist","html_url":"https://github.com/kdahlquist","followers_url":"https://api.github.com/users/kdahlquist/followers","following_url":"https://api.github.com/users/kdahlquist/following{/other_user}","gists_url":"https://api.github.com/users/kdahlquist/gists{/gist_id}","starred_url":"https://api.github.com/users/kdahlquist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kdahlquist/subscriptions","organizations_url":"https://api.github.com/users/kdahlquist/orgs","repos_url":"https://api.github.com/users/kdahlquist/repos","events_url":"https://api.github.com/users/kdahlquist/events{/privacy}","received_events_url":"https://api.github.com/users/kdahlquist/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dondi/GRNsight/labels/feature%20request","name":"feature request","color":"84b6eb"},{"url":"https://api.github.com/repos/dondi/GRNsight/labels/functionality","name":"functionality","color":"0052cc"},{"url":"https://api.github.com/repos/dondi/GRNsight/labels/priority%200","name":"priority 0","color":"f7c6c7"}],"state":"closed","locked":false,"assignee":{"login":"dondi","id":382242,"avatar_url":"https://avatars.githubusercontent.com/u/382242?v=3","gravatar_id":"","url":"https://api.github.com/users/dondi","html_url":"https://github.com/dondi","followers_url":"https://api.github.com/users/dondi/followers","following_url":"https://api.github.com/users/dondi/following{/other_user}","gists_url":"https://api.github.com/users/dondi/gists{/gist_id}","starred_url":"https://api.github.com/users/dondi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dondi/subscriptions","organizations_url":"https://api.github.com/users/dondi/orgs","repos_url":"https://api.github.com/users/dondi/repos","events_url":"https://api.github.com/users/dondi/events{/privacy}","received_events_url":"https://api.github.com/users/dondi/received_events","type":"User","site_admin":false},"assignees":[{"login":"dondi","id":382242,"avatar_url":"https://avatars.githubusercontent.com/u/382242?v=3","gravatar_id":"","url":"https://api.github.com/users/dondi","html_url":"https://github.com/dondi","followers_url":"https://api.github.com/users/dondi/followers","following_url":"https://api.github.com/users/dondi/following{/other_user}","gists_url":"https://api.github.com/users/dondi/gists{/gist_id}","starred_url":"https://api.github.com/users/dondi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dondi/subscriptions","organizations_url":"https://api.github.com/users/dondi/orgs","repos_url":"https://api.github.com/users/dondi/repos","events_url":"https://api.github.com/users/dondi/events{/privacy}","received_events_url":"https://api.github.com/users/dondi/received_events","type":"User","site_admin":false},{"login":"kdahlquist","id":6463867,"avatar_url":"https://avatars.githubusercontent.com/u/6463867?v=3","gravatar_id":"","url":"https://api.github.com/users/kdahlquist","html_url":"https://github.com/kdahlquist","followers_url":"https://api.github.com/users/kdahlquist/followers","following_url":"https://api.github.com/users/kdahlquist/following{/other_user}","gists_url":"https://api.github.com/users/kdahlquist/gists{/gist_id}","starred_url":"https://api.github.com/users/kdahlquist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kdahlquist/subscriptions","organizations_url":"https://api.github.com/users/kdahlquist/orgs","repos_url":"https://api.github.com/users/kdahlquist/repos","events_url":"https://api.github.com/users/kdahlquist/events{/privacy}","received_events_url":"https://api.github.com/users/kdahlquist/received_events","type":"User","site_admin":false}],"milestone":null,"comments":14,"created_at":"2016-08-03T21:33:04Z","updated_at":"2016-08-11T18:24:38Z","closed_at":"2016-08-11T17:56:07Z","body":"I'm creating this issue in parallel to #309 for discussion of GraphML.  I'm going to close #287 and #288 because the basic functionality there has been implemented and we are now fine-tuning.  It makes more sense now to take on the two formats separately.\r\n\r\nI have now read through the GraphML Primer and have done a visual inspection of the GRNsight GraphML exports of an unweighted and weighted graph.\r\n\r\nI can match up the XML elements in the primer with what GRNsight has exported, and just have one question so far:\r\n* Why did you settle on \"edge-value-id\" as the name of the data key for the weight value?  What's throwing me off is the use of the \"id\" in this term.  I have a hard time wrapping my head around a data value being an id.  Shouldn't this be called something like \"edge-weight-value\" or \"edge-value\" without the \"id\"?  Maybe I don't know about a convention you are using?\r\n\r\nI am now proceeding with further testing of *.graphml files moving between programs."},"comment":{"url":"https://api.github.com/repos/dondi/GRNsight/issues/comments/239247405","html_url":"https://github.com/dondi/GRNsight/issues/314#issuecomment-239247405","issue_url":"https://api.github.com/repos/dondi/GRNsight/issues/314","id":239247405,"user":{"login":"dondi","id":382242,"avatar_url":"https://avatars.githubusercontent.com/u/382242?v=3","gravatar_id":"","url":"https://api.github.com/users/dondi","html_url":"https://github.com/dondi","followers_url":"https://api.github.com/users/dondi/followers","following_url":"https://api.github.com/users/dondi/following{/other_user}","gists_url":"https://api.github.com/users/dondi/gists{/gist_id}","starred_url":"https://api.github.com/users/dondi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dondi/subscriptions","organizations_url":"https://api.github.com/users/dondi/orgs","repos_url":"https://api.github.com/users/dondi/repos","events_url":"https://api.github.com/users/dondi/events{/privacy}","received_events_url":"https://api.github.com/users/dondi/received_events","type":"User","site_admin":false},"created_at":"2016-08-11T18:24:38Z","updated_at":"2016-08-11T18:24:38Z","body":"Ah yes forgot to mention---the date/timestamp was tricky because it's hard to unit test. Additional code will need to be written to assert that the comment is present but either ignore a discrepancy in the date/timestamp, or somehow anticipate what the date/timestamp should be (reading the date/timestamp first and then invoking the export is not guaranteed to match because the test might cross a time unit boundary). We can get back to that when there is more elbow room."}}
{"actor":{"display_login":"dondi"}}