Fix Python import system

hiftikha commented 5 months ago

What's New

Fix python import system
Added CI integration for running tests
Resolved dependency conflicts
Removed
Every package follows the following project structure for consistency: package/

package/ ├── init.py ├── src/ │ ├── init.py │ ├── main_module.py │ └── other_modules/ ├── tests/ │ ├── init.py │ ├── main_module_test.py │ ├── other_module_test.py │ └── test_data.py └── README.md

What This Changes

No more hacking needed using sys.path.insert(0, str(Path(__file__).parent.parent)) or sys.path.append(".")
Github Actions will run tests on every PR

Why This Change is Needed

https://www.pivotaltracker.com/story/show/187662597

How to Test

run poetry run pytest

raindrift commented 5 months ago

After merging the PR for code formatting, this one has a lot of conflicts. Which I guess is sort of unavoidable.

raindrift commented 4 months ago

For the moment, it seems like we're happy with how the python imports are working in this repo. I think we might come back to this, but if that's the case it's easy enough to reopen this PR.

HumanCompatibleAI / ranking-challenge