<div dir="ltr"><br><br>2016-10-02 13:24 GMT+02:00 Moritz Lennert <<a href="mailto:mlennert@club.worldonline.be">mlennert@club.worldonline.be</a>>:<br>><br>> On 01/10/16 21:25, Blumentrath, Stefan wrote:<br>>><br>>> Sounds fair enough as requirements for new core modules. “Maintainable<br>>> code” would in praxis mean “the module has undergone a code review by a<br>>> core developer”?<br>>><br>>> Those requirements would add to Markus requirement of “maturity”, which<br>>> I would interpret like “the module has been tested in praxis and options<br>>> and flags are consolidated” (so no major changes are expected /<br>>> planned)...?<br>>><br>>><br>>><br>>> I am afraid, it seems only very few of the suggested modules are covered<br>>> with unit tests. Most of them have a good documentation. No idea about<br>>> the maintainability of the code...<br>>><br>>><br>>><br>>> How should we proceed with this topic? Should the named modules (and<br>>> from my point of view Moritz OBIA modules would be very welcome too)<br>><br>><br>> They definitely do not meet the enounced criteria, yet. No tests and AFAIK, most of them have only been used inhouse by my colleagues.<br>><br>> So, I'm happy to have them live addons for now.<br>><br>> This said, I think the requirement of tests is something I would like to see discussed a bit more. This is a pretty heavy requirement and many current core modules do not have unit tests...<div><br></div><div>You are very welcome to write the missing tests for core modules.<br><br>However, i don't understand the argument that because many core modules have no tests, therefore new modules don't need them. If developers of addon module are serious about the attempt to make their modules usable and maintainable for others, then they have to implement tests. Its and integral part of the development process and GRASS has a beautiful test environment hat makes writing tests easy. Tests and documentation are part of coding and not something special. I don't think this is a hard requirement.</div><div><br></div><div>There is a nice statement that is not far from the truth: Untested code is broken code.<br><br>Best<div>Sören</div><div><br>><br>> One thing we could think about is activating the toolbox idea a bit more and creating a specific OBIA toolbox in addons.<br>><br>>> Identified candidates could be added to core once they fulfill the<br>>> requirements above. Would that happen only in minor releases or would<br>>> that also be possible in point releases?<br>><br>><br>> Adding modules to core is not an API change, so I don't see why they can't be added at any time. But then again, having a series of new modules can be sufficient to justify a new minor release ;-)<br>><br>>> Or is that already too much formality and if someone wishes to see an<br>>> addon in core that is simply discussed on ML?<br>><br>><br>> Generally, I would think that discussion on ML is the best way to handle this.<br>><br>> Moritz<br>><br>><br>> _______________________________________________<br>> grass-dev mailing list<br>> <a href="mailto:grass-dev@lists.osgeo.org">grass-dev@lists.osgeo.org</a><br>> <a href="http://lists.osgeo.org/mailman/listinfo/grass-dev">http://lists.osgeo.org/mailman/listinfo/grass-dev</a></div></div></div>