{"action":"created","issue":{"url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/2053","repository_url":"https://api.github.com/repos/refinery-platform/refinery-platform","labels_url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/2053/labels{/name}","comments_url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/2053/comments","events_url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/2053/events","html_url":"https://github.com/refinery-platform/refinery-platform/issues/2053","id":252948112,"number":2053,"title":"Allow associating uploaded data files with data sets imported from ISA archives","user":{"login":"hackdna","id":452575,"avatar_url":"https://avatars0.githubusercontent.com/u/452575?v=4","gravatar_id":"","url":"https://api.github.com/users/hackdna","html_url":"https://github.com/hackdna","followers_url":"https://api.github.com/users/hackdna/followers","following_url":"https://api.github.com/users/hackdna/following{/other_user}","gists_url":"https://api.github.com/users/hackdna/gists{/gist_id}","starred_url":"https://api.github.com/users/hackdna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hackdna/subscriptions","organizations_url":"https://api.github.com/users/hackdna/orgs","repos_url":"https://api.github.com/users/hackdna/repos","events_url":"https://api.github.com/users/hackdna/events{/privacy}","received_events_url":"https://api.github.com/users/hackdna/received_events","type":"User","site_admin":false},"labels":[{"id":80396991,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/dataset","name":"dataset","color":"ededed","default":false},{"id":80396847,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/enhancement","name":"enhancement","color":"034380","default":true},{"id":80397013,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/files","name":"files","color":"ededed","default":false},{"id":228840735,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/import","name":"import","color":"eeeeee","default":false},{"id":80396992,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/metadata","name":"metadata","color":"ededed","default":false},{"id":252297690,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/server","name":"server","color":"555555","default":false},{"id":80396959,"url":"https://api.github.com/repos/refinery-platform/refinery-platform/labels/ui","name":"ui","color":"555555","default":false}],"state":"open","locked":false,"assignee":{"login":"hackdna","id":452575,"avatar_url":"https://avatars0.githubusercontent.com/u/452575?v=4","gravatar_id":"","url":"https://api.github.com/users/hackdna","html_url":"https://github.com/hackdna","followers_url":"https://api.github.com/users/hackdna/followers","following_url":"https://api.github.com/users/hackdna/following{/other_user}","gists_url":"https://api.github.com/users/hackdna/gists{/gist_id}","starred_url":"https://api.github.com/users/hackdna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hackdna/subscriptions","organizations_url":"https://api.github.com/users/hackdna/orgs","repos_url":"https://api.github.com/users/hackdna/repos","events_url":"https://api.github.com/users/hackdna/events{/privacy}","received_events_url":"https://api.github.com/users/hackdna/received_events","type":"User","site_admin":false},"assignees":[{"login":"hackdna","id":452575,"avatar_url":"https://avatars0.githubusercontent.com/u/452575?v=4","gravatar_id":"","url":"https://api.github.com/users/hackdna","html_url":"https://github.com/hackdna","followers_url":"https://api.github.com/users/hackdna/followers","following_url":"https://api.github.com/users/hackdna/following{/other_user}","gists_url":"https://api.github.com/users/hackdna/gists{/gist_id}","starred_url":"https://api.github.com/users/hackdna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hackdna/subscriptions","organizations_url":"https://api.github.com/users/hackdna/orgs","repos_url":"https://api.github.com/users/hackdna/repos","events_url":"https://api.github.com/users/hackdna/events{/privacy}","received_events_url":"https://api.github.com/users/hackdna/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/refinery-platform/refinery-platform/milestones/47","html_url":"https://github.com/refinery-platform/refinery-platform/milestone/47","labels_url":"https://api.github.com/repos/refinery-platform/refinery-platform/milestones/47/labels","id":2540663,"number":47,"title":"Next","description":"Staging for issues to be included in the next release","creator":{"login":"scottx611x","id":5629547,"avatar_url":"https://avatars2.githubusercontent.com/u/5629547?v=4","gravatar_id":"","url":"https://api.github.com/users/scottx611x","html_url":"https://github.com/scottx611x","followers_url":"https://api.github.com/users/scottx611x/followers","following_url":"https://api.github.com/users/scottx611x/following{/other_user}","gists_url":"https://api.github.com/users/scottx611x/gists{/gist_id}","starred_url":"https://api.github.com/users/scottx611x/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scottx611x/subscriptions","organizations_url":"https://api.github.com/users/scottx611x/orgs","repos_url":"https://api.github.com/users/scottx611x/repos","events_url":"https://api.github.com/users/scottx611x/events{/privacy}","received_events_url":"https://api.github.com/users/scottx611x/received_events","type":"User","site_admin":false},"open_issues":33,"closed_issues":4,"state":"open","created_at":"2017-05-24T13:10:34Z","updated_at":"2017-09-07T21:16:29Z","due_on":null,"closed_at":null},"comments":4,"created_at":"2017-08-25T16:07:33Z","updated_at":"2017-09-07T21:04:10Z","closed_at":null,"author_association":"COLLABORATOR","body":"Currently, ISA archives must reference data files by URLs for successful import into Refinery. We need to allow uploading data files during ISA archive import and associating these data files with the resulting data set.\r\n\r\nThis requires:\r\n* adding file upload functionality to the ISA-Tab import form (POSIX file system upload on Vagrant and S3 upload on AWS)\r\n* changing ISA archive import process to allow adding files from the upload area on S3 (AWS) or from the data file import directory (Vagrant)\r\n\r\nTechnical debt:\r\nrefactor `parse_isatab()` to return data set UUID only (other return items are not actually used)\r\n"},"comment":{"url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/comments/327927120","html_url":"https://github.com/refinery-platform/refinery-platform/issues/2053#issuecomment-327927120","issue_url":"https://api.github.com/repos/refinery-platform/refinery-platform/issues/2053","id":327927120,"user":{"login":"hackdna","id":452575,"avatar_url":"https://avatars0.githubusercontent.com/u/452575?v=4","gravatar_id":"","url":"https://api.github.com/users/hackdna","html_url":"https://github.com/hackdna","followers_url":"https://api.github.com/users/hackdna/followers","following_url":"https://api.github.com/users/hackdna/following{/other_user}","gists_url":"https://api.github.com/users/hackdna/gists{/gist_id}","starred_url":"https://api.github.com/users/hackdna/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hackdna/subscriptions","organizations_url":"https://api.github.com/users/hackdna/orgs","repos_url":"https://api.github.com/users/hackdna/repos","events_url":"https://api.github.com/users/hackdna/events{/privacy}","received_events_url":"https://api.github.com/users/hackdna/received_events","type":"User","site_admin":false},"created_at":"2017-09-07T21:04:10Z","updated_at":"2017-09-07T21:04:10Z","author_association":"COLLABORATOR","body":"This requires a rewrite of `TakeOwnershipOfPublicDatasetView` to perform a deep copy of the imported data set instead of re-importing the original ISA archive since:\r\n* the ISA archive only has references to the uploaded file names and not the imported files\r\n* `ProcessISATabView` doesn't and shouldn't care about the context of the ISA archive import (regular import vs taking ownership)\r\n\r\nIn addition, `TakeOwnershipOfPublicDatasetView` is missing support for data sets imported from tabular files.\r\n\r\nSo, I'm going to submit a pull request covering the original spec for allowing file uploads with ISA archive imports and create a separate issue for updates to `TakeOwnershipOfPublicDatasetView`.\r\n"}}
{"actor":{"display_login":"hackdna"}}