New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

set up rl-human-prefs #4

Closed

yamatokataoka opened this issue Dec 20, 2021 · 5 comments · Fixed by #9

Assignees

Milestone

Owner

yamatokataoka commented Dec 20, 2021 •

edited

Loading

add Python project
set up linter, type checker

yamatokataoka added this to the v0.0.1 MVP milestone

yamatokataoka self-assigned this

Owner Author

yamatokataoka commented Jan 15, 2022

poetry would be good option for sophisticated dependency management

https://mitelman.engineering/blog/python-best-practice/automating-python-best-practices-for-a-new-project/#fast-track

Owner Author

yamatokataoka commented Jan 15, 2022 •

edited

Loading

poetry with Docker
python-poetry/poetry#1879 (comment)

I don't use Docker because we can't run Docker on Colab and isolation of environment would be enough with poetry and its venv.

Owner Author

yamatokataoka commented Jan 16, 2022

linters

isort
black
flake8
mypy

Owner Author

yamatokataoka commented Jan 22, 2022

pyproject.toml
https://martin-thoma.com/pyproject-toml/

Owner Author

yamatokataoka commented Jan 22, 2022

poetry with tox
python-poetry/poetry#1941

yamatokataoka linked a pull request

that will close this issue

set up rl-human-prefs #9

Merged

yamatokataoka closed this as completed in #9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment