{"action":"created","issue":{"url":"https://api.github.com/repos/biocore/scikit-bio/issues/1157","labels_url":"https://api.github.com/repos/biocore/scikit-bio/issues/1157/labels{/name}","comments_url":"https://api.github.com/repos/biocore/scikit-bio/issues/1157/comments","events_url":"https://api.github.com/repos/biocore/scikit-bio/issues/1157/events","html_url":"https://github.com/biocore/scikit-bio/pull/1157","id":113744844,"number":1157,"title":"use hashable object as positional_metadata column name to store its feature metadata","user":{"login":"RNAer","id":1000056,"avatar_url":"https://avatars.githubusercontent.com/u/1000056?v=3","gravatar_id":"","url":"https://api.github.com/users/RNAer","html_url":"https://github.com/RNAer","followers_url":"https://api.github.com/users/RNAer/followers","following_url":"https://api.github.com/users/RNAer/following{/other_user}","gists_url":"https://api.github.com/users/RNAer/gists{/gist_id}","starred_url":"https://api.github.com/users/RNAer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RNAer/subscriptions","organizations_url":"https://api.github.com/users/RNAer/orgs","repos_url":"https://api.github.com/users/RNAer/repos","events_url":"https://api.github.com/users/RNAer/events{/privacy}","received_events_url":"https://api.github.com/users/RNAer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-10-28T05:04:50Z","updated_at":"2015-11-06T18:44:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/biocore/scikit-bio/pulls/1157","html_url":"https://github.com/biocore/scikit-bio/pull/1157","diff_url":"https://github.com/biocore/scikit-bio/pull/1157.diff","patch_url":"https://github.com/biocore/scikit-bio/pull/1157.patch"},"body":"@jairideout , @ebolyen , or @gregcaporaso , mind taking a quick look at this?"},"comment":{"url":"https://api.github.com/repos/biocore/scikit-bio/issues/comments/154498259","html_url":"https://github.com/biocore/scikit-bio/pull/1157#issuecomment-154498259","issue_url":"https://api.github.com/repos/biocore/scikit-bio/issues/1157","id":154498259,"user":{"login":"RNAer","id":1000056,"avatar_url":"https://avatars.githubusercontent.com/u/1000056?v=3","gravatar_id":"","url":"https://api.github.com/users/RNAer","html_url":"https://github.com/RNAer","followers_url":"https://api.github.com/users/RNAer/followers","following_url":"https://api.github.com/users/RNAer/following{/other_user}","gists_url":"https://api.github.com/users/RNAer/gists{/gist_id}","starred_url":"https://api.github.com/users/RNAer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/RNAer/subscriptions","organizations_url":"https://api.github.com/users/RNAer/orgs","repos_url":"https://api.github.com/users/RNAer/repos","events_url":"https://api.github.com/users/RNAer/events{/privacy}","received_events_url":"https://api.github.com/users/RNAer/received_events","type":"User","site_admin":false},"created_at":"2015-11-06T18:44:38Z","updated_at":"2015-11-06T18:44:38Z","body":"fair enough. thanks.\n\nOn Fri, Nov 6, 2015 at 10:39 AM, Evan Bolyen <notifications@github.com>\nwrote:\n\n> @jairideout <https://github.com/jairideout> and I have been discussing\n> this PR. Based on the discussion in #1159\n> <https://github.com/biocore/scikit-bio/issues/1159> it's not clear how we\n> ultimately want to handle sequence features in scikit-bio, so the hashable\n> features may not even be how we actually want to solve this problem.\n> Introducing hashable features is an API change that may not make sense to\n> make right now. If you wanted to include this as part of your fork while\n> you develop a more robust solution that is totally fine. But given the\n> subpar performance of SparseDataFrame it probably isn't worth making this\n> change now, as we need a different API solution for these features anyways.\n>\n> I'm also not convinced that using SparseDataFrame will work either, as I\n> expect that slicing the sequence by a SparseSeries in the SparseDataFrame\n> will produce an incorrect sequence.\n>\n> As for #1167 <https://github.com/biocore/scikit-bio/issues/1167>, I would\n> open a new PR for that. That functionality is totally fine, and we should\n> be able to review it quickly.\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/biocore/scikit-bio/pull/1157#issuecomment-154496915>.\n>\n"}}