Arulkumar03/Wheat_HEAD_Detection_Counting_ComputerVision_Model

1# Contributing to detectron22 3## Issues4We use GitHub issues to track public bugs and questions.5Please make sure to follow one of the6[issue templates](https://github.com/facebookresearch/detectron2/issues/new/choose)7when reporting any issues.8 9Facebook has a [bounty program](https://www.facebook.com/whitehat/) for the safe10disclosure of security bugs. In those cases, please go through the process11outlined on that page and do not file a public issue.12 13## Pull Requests14We actively welcome pull requests.15 16However, if you're adding any significant features (e.g. > 50 lines), please17make sure to discuss with maintainers about your motivation and proposals in an issue18before sending a PR. This is to save your time so you don't spend time on a PR that we'll not accept.19 20We do not always accept new features, and we take the following21factors into consideration:22 231. Whether the same feature can be achieved without modifying detectron2.24   Detectron2 is designed so that you can implement many extensions from the outside, e.g.25   those in [projects](https://github.com/facebookresearch/detectron2/tree/master/projects).26   * If some part of detectron2 is not extensible enough, you can also bring up a more general issue to27     improve it. Such feature request may be useful to more users.282. Whether the feature is potentially useful to a large audience (e.g. an impactful detection paper, a popular dataset,29   a significant speedup, a widely useful utility),30   or only to a small portion of users (e.g., a less-known paper, an improvement not in the object31   detection field, a trick that's not very popular in the community, code to handle a non-standard type of data)32   * Adoption of additional models, datasets, new task are by default not added to detectron2 before they33     receive significant popularity in the community.34     We sometimes accept such features in `projects/`, or as a link in `projects/README.md`.353. Whether the proposed solution has a good design / interface. This can be discussed in the issue prior to PRs, or36   in the form of a draft PR.374. Whether the proposed solution adds extra mental/practical overhead to users who don't38   need such feature.395. Whether the proposed solution breaks existing APIs.40 41To add a feature to an existing function/class `Func`, there are always two approaches:42(1) add new arguments to `Func`; (2) write a new `Func_with_new_feature`.43To meet the above criteria, we often prefer approach (2), because:44 451. It does not involve modifying or potentially breaking existing code.462. It does not add overhead to users who do not need the new feature.473. Adding new arguments to a function/class is not scalable w.r.t. all the possible new research ideas in the future.48 49When sending a PR, please do:50 511. If a PR contains multiple orthogonal changes, split it to several PRs.522. If you've added code that should be tested, add tests.533. For PRs that need experiments (e.g. adding a new model or new methods),54   you don't need to update model zoo, but do provide experiment results in the description of the PR.554. If APIs are changed, update the documentation.565. We use the [Google style docstrings](https://www.sphinx-doc.org/en/master/usage/extensions/napoleon.html) in python.576. Make sure your code lints with `./dev/linter.sh`.58 59 60## Contributor License Agreement ("CLA")61In order to accept your pull request, we need you to submit a CLA. You only need62to do this once to work on any of Facebook's open source projects.63 64Complete your CLA here: <https://code.facebook.com/cla>65 66## License67By contributing to detectron2, you agree that your contributions will be licensed68under the LICENSE file in the root directory of this source tree.69