{"action":"created","issue":{"url":"https://api.github.com/repos/simphony/simphony-common/issues/91","labels_url":"https://api.github.com/repos/simphony/simphony-common/issues/91/labels{/name}","comments_url":"https://api.github.com/repos/simphony/simphony-common/issues/91/comments","events_url":"https://api.github.com/repos/simphony/simphony-common/issues/91/events","html_url":"https://github.com/simphony/simphony-common/issues/91","id":60524597,"number":91,"title":"Suggestion: Make hdf5 package under io","user":{"login":"mehdisadeghi","id":3101557,"avatar_url":"https://avatars.githubusercontent.com/u/3101557?v=3","gravatar_id":"","url":"https://api.github.com/users/mehdisadeghi","html_url":"https://github.com/mehdisadeghi","followers_url":"https://api.github.com/users/mehdisadeghi/followers","following_url":"https://api.github.com/users/mehdisadeghi/following{/other_user}","gists_url":"https://api.github.com/users/mehdisadeghi/gists{/gist_id}","starred_url":"https://api.github.com/users/mehdisadeghi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mehdisadeghi/subscriptions","organizations_url":"https://api.github.com/users/mehdisadeghi/orgs","repos_url":"https://api.github.com/users/mehdisadeghi/repos","events_url":"https://api.github.com/users/mehdisadeghi/events{/privacy}","received_events_url":"https://api.github.com/users/mehdisadeghi/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/simphony/simphony-common/labels/enhancement","name":"enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/simphony/simphony-common/labels/Low%20Priority","name":"Low Priority","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-03-10T16:09:03Z","updated_at":"2015-09-11T15:09:07Z","closed_at":null,"body":"I suggest to make a new package called `h5` or `pytables` package under `simphony.io` and then move all pytables related files from `io` to the new package. \r\nI also suggest to change module names inside that folder as the following:\r\n\r\n    data_container_description\r\n    data_container_table\r\n    indexed_data_container_table\r\nAll to be moved to a `containers` or similarly named module (one module) or a package.\r\n\r\n    file_mesh -> h5_mesh\r\nEverything could be `h5` prefixed.\r\n\r\nMoreover we could have a convention in favor of shorter names and less modules. And last but not least I could think of an io layer that give us flexibility to change persistence implementation with no code change.\r\n\r\nPlease let me know your ideas."},"comment":{"url":"https://api.github.com/repos/simphony/simphony-common/issues/comments/139570789","html_url":"https://github.com/simphony/simphony-common/issues/91#issuecomment-139570789","issue_url":"https://api.github.com/repos/simphony/simphony-common/issues/91","id":139570789,"user":{"login":"nathanfranklin","id":8287580,"avatar_url":"https://avatars.githubusercontent.com/u/8287580?v=3","gravatar_id":"","url":"https://api.github.com/users/nathanfranklin","html_url":"https://github.com/nathanfranklin","followers_url":"https://api.github.com/users/nathanfranklin/followers","following_url":"https://api.github.com/users/nathanfranklin/following{/other_user}","gists_url":"https://api.github.com/users/nathanfranklin/gists{/gist_id}","starred_url":"https://api.github.com/users/nathanfranklin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nathanfranklin/subscriptions","organizations_url":"https://api.github.com/users/nathanfranklin/orgs","repos_url":"https://api.github.com/users/nathanfranklin/repos","events_url":"https://api.github.com/users/nathanfranklin/events{/privacy}","received_events_url":"https://api.github.com/users/nathanfranklin/received_events","type":"User","site_admin":false},"created_at":"2015-09-11T15:09:07Z","updated_at":"2015-09-11T15:09:07Z","body":"I think we should close this issue for the time being as we only are using one backend (hdf5) at the moment.  Once we want to add another serialization backed, we can reopen this issue."}}