{"action":"created","issue":{"url":"https://api.github.com/repos/neuropoly/qMRLab/issues/175","repository_url":"https://api.github.com/repos/neuropoly/qMRLab","labels_url":"https://api.github.com/repos/neuropoly/qMRLab/issues/175/labels{/name}","comments_url":"https://api.github.com/repos/neuropoly/qMRLab/issues/175/comments","events_url":"https://api.github.com/repos/neuropoly/qMRLab/issues/175/events","html_url":"https://github.com/neuropoly/qMRLab/issues/175","id":301563677,"number":175,"title":"Random things that could/should be fixed in documentation","user":{"login":"ileppe","id":28764708,"avatar_url":"https://avatars0.githubusercontent.com/u/28764708?v=4","gravatar_id":"","url":"https://api.github.com/users/ileppe","html_url":"https://github.com/ileppe","followers_url":"https://api.github.com/users/ileppe/followers","following_url":"https://api.github.com/users/ileppe/following{/other_user}","gists_url":"https://api.github.com/users/ileppe/gists{/gist_id}","starred_url":"https://api.github.com/users/ileppe/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ileppe/subscriptions","organizations_url":"https://api.github.com/users/ileppe/orgs","repos_url":"https://api.github.com/users/ileppe/repos","events_url":"https://api.github.com/users/ileppe/events{/privacy}","received_events_url":"https://api.github.com/users/ileppe/received_events","type":"User","site_admin":false},"labels":[{"id":853164916,"url":"https://api.github.com/repos/neuropoly/qMRLab/labels/Model:dti","name":"Model:dti","color":"d4c5f9","default":false},{"id":699310135,"url":"https://api.github.com/repos/neuropoly/qMRLab/labels/documentation","name":"documentation","color":"006b75","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2018-03-01T21:21:01Z","updated_at":"2018-03-02T19:34:31Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"- [ ] by default `qMRshowOutput` in the batch shows the fit in the center of the middle slice, but this often ends up being in the central sulcus where fits might look very weird e.g.\r\n![screen shot 2018-03-01 at 15 57 22](https://user-images.githubusercontent.com/28764708/36869297-54b27c84-1d69-11e8-9fef-333cf1485e48.png)\r\n- [x] `GenerateDocumentation` should create a `Data` folder if it does not exist\r\n- [x] matlab2014b does not support downloading from OSF @agahkarakuzu I think you are working on this, I am just putting a tick box ;)\r\n- [ ] the range of the figures if often odd, not showing proper contrast e.g. B0map would be more informative if shown from -50,50\r\n![screen shot 2018-03-01 at 16 29 41](https://user-images.githubusercontent.com/28764708/36870677-ddd03156-1d6d-11e8-9b2f-569fb78ee6fb.png)\r\n\r\n\r\nIssues specific to DTI model\r\n- [ ] The header of the model needs to be completed/fixed\r\n- [ ] the sample data provided for `dti_batch` is overkill for this model (i.e. 109 frames when really only 6 are needed) (and the fit takes forever)\r\n- [ ] The default map that shows up on the documentation is `residue`, I think `FA` would be more informative (`xnames` should include all the outputs i.e. `FA`, `MD`,`residue`, etc right now `xnames = { 'L1','L2','L3'};` )\r\n"},"comment":{"url":"https://api.github.com/repos/neuropoly/qMRLab/issues/comments/370028957","html_url":"https://github.com/neuropoly/qMRLab/issues/175#issuecomment-370028957","issue_url":"https://api.github.com/repos/neuropoly/qMRLab/issues/175","id":370028957,"user":{"login":"agahkarakuzu","id":9632322,"avatar_url":"https://avatars3.githubusercontent.com/u/9632322?v=4","gravatar_id":"","url":"https://api.github.com/users/agahkarakuzu","html_url":"https://github.com/agahkarakuzu","followers_url":"https://api.github.com/users/agahkarakuzu/followers","following_url":"https://api.github.com/users/agahkarakuzu/following{/other_user}","gists_url":"https://api.github.com/users/agahkarakuzu/gists{/gist_id}","starred_url":"https://api.github.com/users/agahkarakuzu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/agahkarakuzu/subscriptions","organizations_url":"https://api.github.com/users/agahkarakuzu/orgs","repos_url":"https://api.github.com/users/agahkarakuzu/repos","events_url":"https://api.github.com/users/agahkarakuzu/events{/privacy}","received_events_url":"https://api.github.com/users/agahkarakuzu/received_events","type":"User","site_admin":false},"created_at":"2018-03-02T19:34:31Z","updated_at":"2018-03-02T19:34:31Z","author_association":"NONE","body":"@ileppe if there is not any difference between the @HarrisNami 's R2014b and others then yes :)\r\nver>R2014b must be satisfied. \r\n\r\n`GenerateDocumentation` will be freed from Data folder . We will work on this with @gab-berest on Monday. \r\n\r\nYour xnames suggestion is now under [wiki page](https://github.com/neuropoly/qMRLab/wiki/Guideline:-Adding-a-new-Model#-property-2-xnames). Although it is specific to dti model, gives clue about the issue.\r\n"}}
{"actor":{"display_login":"agahkarakuzu"}}