CONTRIBUTING.md

How to contribute

Thanks for taking the time to contribute to tiny-dnn! The following is a few guidelines for contributors. These are just guidelines, not rules, and feel free to propose changes to this document in a pull request.

Getting Started

• Make sure you have a C++11 compiler.
• Make sure you have a GitHub account.
• Register a report about your issue.
  • Check the issue list to see if the problem has already been reported.
  • This can be skipped if the issue is trivial (fixing a typo, etc).

Making Changes

• Create a topic branch where you want to base your work.
  • This is usually the master branch.
• Make commits.
• Make sure you have added the necessary tests for your changes.
• Submit a pull request.
• Make sure all CI builds are passed.

Coding guides

• Keep header-only
• Keep dependency-free
  • If your change requires 3rd party libraries, this should be optional in tiny-dnn. Please guard your 3rd party dependent code by #ifdef - #endif block, and write CMakelist option to enable the block - but lesser these switches, the better.
• Keep platform-independent
  • Use C++ standard library instead of Windows/POSIX dependent API
  • CPU/GPU optimized code should be extracted as a separated file, and should be guarded as preprocessor macro.

Preferred coding style

• Use Google coding style guide with some exceptions:
  • Use CNN_NAME_OF_THE_MACRO style for preprocessor macros.
  • Use snake_case for rest of identifiers
  • "We do not use C++ exceptions" - We are using exceptions which throw tiny_dnn::nn_error or its subclass to keep error handling simple.
  • "Avoid using Run Time Type Information (RTTI)" - We are using RTTI for serialization/deserialization.
  • "All parameters passed by reference must be labeled const" - We sometimes use non-const reference to 1) avoid null-pointer dereference, or 2) keep code clean (especially when overloading operator << (std::ostream&,T)
  • "All header files should have #define guards to prevent multiple inclusion" - We are using #pragma once because include guards are error-prone. It is implementation defined, but many compilers support it.