-
Notifications
You must be signed in to change notification settings - Fork 26
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* added CONTRIBUTING.md * updates to docs, tests, and README
- Loading branch information
1 parent
76b13ca
commit 39bf22a
Showing
71 changed files
with
95 additions
and
26,915 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,55 @@ | ||
# Contributing to TRAK | ||
|
||
Thanks for your interest in contributing to TRAK! We welcome any sort of | ||
feedback---bug fixes, questions, extensions, etc. | ||
|
||
## Extending TRAK | ||
|
||
If you have extended TRAK to your own task (e.g., by subclassing | ||
`AbstractModelOutput` or `AbstractGradientComputer`), you can make a pull | ||
request to add your extension to the `trak/contrib` directory. Then, users | ||
can use your extension by importing it from `trak.contrib`. Below, we provide | ||
guidelines for how to structure your extension. | ||
|
||
1. Create a new directory in `trak/contrib` for your extension. For example, | ||
if you are extending TRAK to work with diffusion models, you might create | ||
a directory called `diffusion_models`. | ||
|
||
2. Create a `README.md` file in your new directory. This file should contain | ||
a description of your extension and a brief example of how to use it. | ||
|
||
3. Add all modules that implement your extension. | ||
|
||
4. If your extension requires any dependencies that are not already listed in | ||
`setup.py`, add an entry to the `extras_require` dictionary in `setup.py`. | ||
For example, if your extension requires `diffusers`, you might add the | ||
following: `'diffusion_models': ['diffusers']`. Then, users can install your | ||
extension's dependencies with `pip install traker[diffusion_models]`. Do | ||
**not** add the dependencies to the `install_requires` list in `setup.py`. | ||
|
||
5. Add any tests in a subdirectory of `tests/contrib` matching the name of your | ||
extension's directory. For example, if your extension is in | ||
`trak/contrib/diffusion_models`, add tests in | ||
`tests/contrib/diffusion_models`. At a minimum, submit an integration test | ||
that demonstrates how to use your extension. Ideally, also submit unit tests | ||
that verify that your extension works as expected. | ||
|
||
6. Once you are done, make a pull request to add your extension to TRAK. | ||
|
||
## Bugs | ||
|
||
If you observe a bug, make an issue with a code snippet that reproduces the | ||
undesired behavior. Feel free to make pull requests that address the bug (see | ||
below). | ||
|
||
## Bug fixes | ||
|
||
If you observe a bug, and you know how to fix it, feel free to make a pull | ||
request that fixes the bug. Please include a unit test that demonstrates the | ||
bug and verifies that your fix works. Additionally, please run the existing | ||
tests to ensure that your fix does not break any existing functionality. You can | ||
install the test dependencies with `pip install traker[tests]`. | ||
|
||
Note that some of the tests are compute-intensive and may require a GPU to run. | ||
So long as your fix does not interact at all with the functionality being tested | ||
by these tests, you can skip them by running `pytest -m "not cuda"`. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
# Sphinx build info version 1 | ||
# This file hashes the configuration used when building these files. When it is not found, a full rebuild will be done. | ||
config: e60030996305630cdf692b57404fa4b3 | ||
config: 746b6350ce2007ec99ab115c5cdc39c4 | ||
tags: 645f666f9bcd5a90fca523b33c5a78b7 |
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.