Skip to content

Conversation

jlamypoirier
Copy link
Collaborator

@jlamypoirier jlamypoirier commented Jan 13, 2023

Requirements, submodule, dockerfile (with specified used so we don't have to run as root), formatting with black/isort (copied from transformers repo)

@jlamypoirier jlamypoirier changed the title Add dockerfile and requirements Add dockerfile, requirements and code stylel Jan 13, 2023
@jlamypoirier jlamypoirier changed the title Add dockerfile, requirements and code stylel Add dockerfile, requirements and code style Jan 13, 2023
@mayank31398
Copy link
Collaborator

Haven't tried this yet since I don't have an environment to run docker containers with GPUs unfortunately.
But the changes look good to me.

@mayank31398 mayank31398 merged commit 36cacf0 into main Jan 14, 2023
@mayank31398 mayank31398 deleted the dockerfile branch January 14, 2023 01:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants