marian/CONTRIBUTING.md
2017-10-15 14:34:52 +00:00

2.2 KiB

How to contribute to Marian

Reporting a bug/asking questions

Did you find a bug or want to ask a question? Great!

  • Ensure the bug was not already reported or the question was not asked
  • For bugs check the github issues
  • For answers to your question search posts on the Google discussion group
  • Open a new issue/question if you're unable to find yours
  • For bugs please provide as much relevant information as possible, and do not forget to paste a log

You are also welcome to request a new feature. Especially if you plan to help us adding it :)

Submitting changes

Whenever possible, please send a Github Pull Request with a clear list of what you've done. Feel free to also update CHANGELOG.md file. We will love you forever if you provide unit or regression tests.

Please follow our coding convention (below) and make sure all of your commits are atomic (learn more about git squash to merge multiple commits and git rebase -i to split a single huge commit into smaller pieces).

The most recent branch to start working with usually is the develop branch. The changes from develop are merged with master, and after that the VERSION file should be updated.

Ideally test your changes by running Marian regression tests locally:

git clone http://github.com/marian-nmt/marian-dev-regression-tests.git
cd marian-dev-regression-tests.git
make BRANCH=<your_branch_name> install
./run_mrt.sh

Coding conventions

Main code style rules:

  • no tabs, 2 whitespaces instead
  • lines no longer than 80 characters
  • no trailing whitespaces
  • no space between control statements and opening brackets
  • UpperCamelCase for class names
  • camelCaseWithTrailingUnderscore_ for class variables
  • camelCase for variables, methods and functions
  • UPPERCASE_WITH_UNDERSCORES for constants

Ideally, use the provided .clang-format file (in the root directory) for ClangFormat to format your code, e.g.

clang-format-3.8 <path_to_file>