ensmallen is an open-source project, so contributions are welcome and encouraged! Anyone can join the ensmallen developer community simply by opening a pull request with an improvement, bugfix, or feature addition, and an ensmallen maintainer will review it and help through the merge process. So if you have an improvement you would like to see, we would love to include it!
The ensmallen maintainer community overlaps heavily with the mlpack community, so development discussions can happen either here on Github, on the mlpack mailing list, or in the #mlpack IRC channel on irc.freenode.net.
Once a pull request is submitted, it must be reviewed and approved before a merge, to ensure that:
- the design matches the rest of the ensmallen design
- the style matches the mlpack style guide
- any new functionality is tested and working
Please do make sure that if you contribute a new optimizer or other new
functionality, that you've added some tests in the tests/ directory. And if
you are fixing a bug, it's always nice to include a test case if possible to so
that the bug won't happen again.
All of the code for ensmallen is located in include/ensmallen_bits/ and all of
the tests are in tests/. ensmallen is header-only, so anything in
include/ensmallen_bits/ must be either template methods or marked inline.
Adding a new test can be done either by adding a new TEST_CASE() block to an
existing file in tests/ or by creating a new file and adding it to the list of
test sources in tests/CMakeLists.txt. The tests are written with the
Catch2 unit test framework.
Sometimes, tests have random behavior and may not pass every time. (For
instance, consider a test where the initial point is randomly generated.) If
you have written a test like this, make sure it does not fail often by
uncommenting the code that sets a random seed in tests/main.cpp and running
your test many times.
Information on how to build and run the tests is in the main README.md file.