Skip to content

Latest commit

 

History

History
28 lines (14 loc) · 2.51 KB

CONTRIBUTING.md

File metadata and controls

28 lines (14 loc) · 2.51 KB

How to contribute to MulensModel?

Any contribution to the MulensModel package is welcomed, even if it is just another test or example on how to use MulensModel. We note contributions in AUTHORS.md file.

Have you found a bug?

If you see a bug in the code or would like to see a new feature, then please open a new issue. If you don't think an issue is the right category, then start a new discussion.

Would you like to see a new feature?

We're also open to suggestions, new ideas, and requests. If you have them, then just open a new issue. We have a long list of changes that we would like to make and your feedback may help us prioritize them. A fresh look at the code is always useful!

Do you have code to contribute?

Having a code that you want to share is great! We follow coding conventions (e.g., PEP8, sphinx-compatible docstrings for public functions, camel case names of classes, snake case names of attributes and methods, etc.) but you don't have to follow all of them right away. Just open a new pull request and we will have some discussion on changes that need to be done before merging. Please note that we expect that you will provide a unit test for every new public function that is not plotting. If the function is plotting, then we expect a new example. Sometimes there is significant discussion befor the pull request is accepted (see some closed ones for examples).

An important aspect of adding new code is writing tests that can be run in automated way. The tests are in source/MulensModel/tests/ directory and you can run them by invoking pytest command (it requires pytest package to be installed). If you don't know how to write tests for your code, don't worry - we will help you. We expect that new code is tested in nearly 100% when it's merged.

Weekly hack sessions

As of Nov 2022, we have regular hack sessions (just 2h each week) to work on MulensModel. Contact us if you are interested in joining.

Thanks!

Radek & Jennifer