oesteban / diffantom

A whole-brain diffusion phantom using phantomas
MIT License
2 stars 1 forks source link

General thread for review #10

Closed oesteban closed 9 years ago

oesteban commented 9 years ago

The paper: https://www.dropbox.com/s/v9r0yvh4tdag4iz/20150715-DiffantomV1.0.pdf.

Updated link to paper: https://www.dropbox.com/s/1ec132dhqmel4n7/20150715-DiffantomV1.1.pdf

oesteban commented 9 years ago

Added the video: https://www.dropbox.com/s/42wjngd1rpb2ie3/20150715-DiffantomVideo.mp4?dl=0

For the video: ffmpeg -f image2 -r 30 -i 'myanimation.%04d.png' 20150715-DiffantomVideo.mp4

oesteban commented 9 years ago

I've received a review from Gert:

I have not much say about the content, it looks like e nice paper to me and is generally well written. I'd only suggest some miner corrections:

Generally, you should not split sentences by long lists of references, this is okay for numbers, but makes reading the article difficult. In these cases it is best to move the references to the end of the phrase. (e.g. phrases starting at 31.. 33..) 1: <...> application for the in-vivo <...> 5: <...> sclerosis, or Alzheimer's <...>

Switch order of sentence: 29 - 31: Digital simulations are increasingly popular because the complexity of <...> tractography can not be accounted for with <...>. 31: Early digital phantoms also started with simulation of simple geometries <...> 50: The simulation process is started by generating <...> 57: The process to obtain these inputs is as follows: <...> or The process to obtain these inputs can be described like follows: <...> 58-64: You should stick to either passive or active voice for all the points 1-5. 107: <...> that is highly demanding <...> 108: Lastly -> Finally

Hope that helps & good luck, Gert

oesteban commented 9 years ago

Changes from @asantos5 are in this commit: https://github.com/oesteban/diffantom/commit/1fa0459ea5f7612f1d9517c7ffefa6c4355a61e4

Changes from @meribach : https://github.com/oesteban/diffantom/commit/6e3645fd33a9ebbf048585f9fb0633e82f5858c0