DorianDepriester / MTEX2Gmsh

Matlab toolbox for generating 2D meshes from EBSD data
https://doriandepriester.github.io/MTEX2Gmsh/
MIT License
41 stars 18 forks source link

Documentation #1

Closed streeve closed 4 years ago

streeve commented 4 years ago

I have a few suggestions to improve documentation per openjournals/joss-reviews#2094

Concerning the tutorial:

The example cases are easy to follow and seem to cover the extent of the intended uses, but unit testing is missing. Some recently accepted joss papers using matlab have what seems to be an offline-automated test suite

I did not see any API documentation (maybe I am forgetting how to access this in matlab)

Finally, make sure to add a CONTRIBUTING.md file for Github contributors

DorianDepriester commented 4 years ago

Thank you @streeve for your quick review. All the points you mentioned have been taken into consideration.

Just to mention, the unit test are not that easy to implement on MATLAB. As far as I know, Code Ocean is the easiest way to provide offline test for reproducibility. The link to the related Capsule has been added in README.md.

Considering the API docs, I am not sure to understand its meaning in the MATLAB scripting context. All functions and methods are provided along with help sections (in addition to documentation). For instance, juste type

help gmshGeo

to get help about the gmshGeo constructor.

streeve commented 4 years ago

Great, thanks

The Code Ocean online environment is quite nice. My only objection is that (as far as I can tell) it only tests one case. Something like https://github.com/vitorsr/SIHR/blob/master/utils/automated_testing.m could work

streeve commented 4 years ago

For the API docs, a section just mentioning both help (thank you, I haven't actively used matlab in a while) and methods to make it easy to explore all options in the class

DorianDepriester commented 4 years ago

I have added a file named unitTest. It sums up all the examples given in the eponymous folder, plus it plots the geometry in each case.

DorianDepriester commented 4 years ago

Alternatively, I have update the code on code Ocean, following your recommendations (that is, provide multiple examples).

Tell me if if need to keep the aforementioned unitTest.m or/and the reference to the capsule.

streeve commented 4 years ago

I think unitTest.m is exactly what you want: it runs the main examples, is easy to extend, and you can run on Code Ocean or locally