{"action":"created","issue":{"url":"https://api.github.com/repos/jhprinz/msm-tis/issues/5","labels_url":"https://api.github.com/repos/jhprinz/msm-tis/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/jhprinz/msm-tis/issues/5/comments","events_url":"https://api.github.com/repos/jhprinz/msm-tis/issues/5/events","html_url":"https://github.com/jhprinz/msm-tis/pull/5","id":56994257,"number":5,"title":"[WIP] Add hidden MovePath capabilities","user":{"login":"jhprinz","id":5091934,"avatar_url":"https://avatars.githubusercontent.com/u/5091934?v=3","gravatar_id":"","url":"https://api.github.com/users/jhprinz","html_url":"https://github.com/jhprinz","followers_url":"https://api.github.com/users/jhprinz/followers","following_url":"https://api.github.com/users/jhprinz/following{/other_user}","gists_url":"https://api.github.com/users/jhprinz/gists{/gist_id}","starred_url":"https://api.github.com/users/jhprinz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhprinz/subscriptions","organizations_url":"https://api.github.com/users/jhprinz/orgs","repos_url":"https://api.github.com/users/jhprinz/repos","events_url":"https://api.github.com/users/jhprinz/events{/privacy}","received_events_url":"https://api.github.com/users/jhprinz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-02-09T06:33:44Z","updated_at":"2015-02-10T17:23:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jhprinz/msm-tis/pulls/5","html_url":"https://github.com/jhprinz/msm-tis/pull/5","diff_url":"https://github.com/jhprinz/msm-tis/pull/5.diff","patch_url":"https://github.com/jhprinz/msm-tis/pull/5.patch"},"body":"This adds the proposed way to use open and closed movepaths. Works nicely with storage.\r\n\r\nAll it does is that\r\n\r\n`movepath.closed` replaces the MovePath by a CollapsedMovePath object and `movepath.opened` returns the old object. If you store a closed object then the loaded one will not allow to be opened. For example this way\r\nthe MinusMover only stores 2 samples now. And it can be loaded in the usual manner.\r\n\r\nIf you have access to a movepath that has been created by a MinusMove and is not loaded you can expand it back using `movepath.opened`\r\n\r\n#### Example\r\n```\r\nprint production_sampleset[0].movepath\r\n                RandomChoice :\r\n                 +- MinusMover [collapsed] : 2 samples [<Sample @ 0x10d9a1fd0>, <Sample @ 0x10d9a1e90>]\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/jhprinz/msm-tis/issues/comments/73742209","html_url":"https://github.com/jhprinz/msm-tis/pull/5#issuecomment-73742209","issue_url":"https://api.github.com/repos/jhprinz/msm-tis/issues/5","id":73742209,"user":{"login":"jhprinz","id":5091934,"avatar_url":"https://avatars.githubusercontent.com/u/5091934?v=3","gravatar_id":"","url":"https://api.github.com/users/jhprinz","html_url":"https://github.com/jhprinz","followers_url":"https://api.github.com/users/jhprinz/followers","following_url":"https://api.github.com/users/jhprinz/following{/other_user}","gists_url":"https://api.github.com/users/jhprinz/gists{/gist_id}","starred_url":"https://api.github.com/users/jhprinz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhprinz/subscriptions","organizations_url":"https://api.github.com/users/jhprinz/orgs","repos_url":"https://api.github.com/users/jhprinz/repos","events_url":"https://api.github.com/users/jhprinz/events{/privacy}","received_events_url":"https://api.github.com/users/jhprinz/received_events","type":"User","site_admin":false},"created_at":"2015-02-10T17:23:46Z","updated_at":"2015-02-10T17:23:46Z","body":"Great. Let's maybe sort this out next week, when I am back? Then we are on the same time and it should be easier.\r\n\r\nJust added a decorator to enable this feature like\r\nmover = MinusMover(..., keep_samples = [2,4])\r\n\r\nThis will just add a FilterSamplesMovePath that only keeps sample at index 2,4 from all the ones returned. I will not use it, but leave it in the code.\r\n\r\nWill check that travis builds and then merge and send you the storage query hopefully still today.\r\nWas good talking to you!"}}