Skip to content

Commit 8d3e3eb

Browse files
janeyx99pytorchmergebot
authored andcommitted
Add note about testing inconsistency
Mitigates pytorch#56396 The true fix would be pytorch#75608 but there are many issues to debug and this is a good step forward to at least clarify. Pull Request resolved: pytorch#75620 Approved by: https://github.com/kit1980, https://github.com/seemethere, https://github.com/malfet
1 parent 6d9210b commit 8d3e3eb

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

CONTRIBUTING.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -342,6 +342,8 @@ The `expecttest` and `hypothesis` libraries must be installed to run the tests.
342342
an optional dependency, and `pytest` may help run tests more selectively.
343343
All these packages can be installed with `conda` or `pip`.
344344

345+
**Weird note:** In our CI (Continuous Integration) jobs, we actually run the tests from the `test` folder and **not** the root of the repo, since there are various dependencies we set up for CI that expects the tests to be run from the test folder. As such, there may be some inconsistencies between local testing and CI testing--if you observe an inconsistency, please [file an issue](https://github.com/pytorch/pytorch/issues/new/choose).
346+
345347
### Better local unit tests with `pytest`
346348

347349
We don't officially support `pytest`, but it works well with our

0 commit comments

Comments
 (0)