[x] p.1 l.13: "Helmholtz" is written twice, and "1856" three times
[x] p.1 l.44: "more practical utility" understates goal of the paper, rephrase/elaborate
Zen: "illusions can serve a more practical utility" to "illusions are a fundamental basis for several scientific explorations"
[x] p.2 l.19-23: add transition from sentence regarding neurocognitive disturbances (no ref attached) to next line
Zen: started paragraph with: 'These neurocognitive deficits have been widely investigated using visual illusions as illusory paradigms demonstrate different contextual influences on visual perception. Visual illusions are thus valuable tools for revealing core features of pathological conditions, such as atypical integration processes in schizophrenia.'
[x] p.6 l.44-55: at least three things missing in the caption (“illusion strength”, “difference”, and “size of the smallest rod”)
Zen: this was a problem with the knitted markdown - somehow \textit{} (to italicize captions) works with pdf but not docx. need to investigate
[x] p.10: names of different illusions sometimes overlap with the illusions.
Zen: hard to fit all in one image, try generating again
[x] Page 11 line 38 – syntax error: parameters = ill.(illusion_strength=2) Missing function name after ill.
[x] Clarify the units images are generated in and if there is flexibility to use different units (this could do with adding to the docstrings)
Zen: Added sentence "The final images are rendered at 800 x 600 pixels (default)". We also already mention in our manuscript that images can be easily post-processed using the PIL library. Added a sentence after: "For example, to see what other image file formats can be saved, refer to PIL's documentation. Will mention these in the docstrings too - in addition to the possibility of resizing the images using the PIL library - resizing is arguably less important in this context (and hence not mentioned in manuscript) as it is unlikely to happen especially when images are generated on the go in an experimental setting.
[ ] update release version!!
[ ] Page 18 line 27: Since porting pyllusion to javascript might be a much longer term goal (and it is challenging to port whole python libs to JS!) it might be worth highlighting that one could easily use the pyllusion generated .png images and make the wider framework of the experiment in a software like PsychoPy builder to use the Pyllusion generated stimuli online.
It's true that porting Python dependencies to JS is an issue. That said, it might be feasible to add directly a JS extension that converts the parameters into the physical properties (as this is essentially basic maths operations). Then, drawing these primitives would require some extensions indeed.
parameters = ill.(illusion_strength=2)
Missing function name after ill.