#2469: None oft the thorns in WVUThorns_Diagnostics have documentation
Reporter:Roland Haas
Status:new
Milestone:
Version:
Type:bug
Priority:major
Component:EinsteinToolkit thorn

All thorns in the WVUThorns_Diagnostics arrangement are lacking a file documentation.tex and thus

Having documentation is a requirement for inclusion in the ET (http://einsteintoolkit.org/contribute.html)

* Components should be of sufficient quality to be used for peer-reviewed and published science. This includes a basic standard of software engineering, documentation of the software including algorithms and methods, tutorials and examples, and self-tests to demonstrate that the software works correctly.

The source code uses an misleading extension for C source files namely it used .C (uppercase letter “c”) which in GNU make (and Cactus) is used for C++ files so is compiled with g++ (but is not recommended since it assumes a file system that distinguishes and preserves case): https://www.gnu.org/software/make/manual/html_node/Catalogue-of-Rules.html#index-_002eC None of the files seem to actually be C++ code eg VolumeIntegrals_GRMHD/src/perform_integration_sum_global.C contains void VI_GRMHD_DoSum(CCTK_ARGUMENTS) which is not typically used in C++ code (though the required extern “C” is already provided by Cactus' prototype (so that it is actually found by schedule.ccl).

--
Ticket URL: https://bitbucket.org/einsteintoolkit/tickets/issues/2469/none-oft-the-thorns-in