Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Parlai container #4868

Merged
merged 5 commits into from
Nov 10, 2022
Merged

Parlai container #4868

merged 5 commits into from
Nov 10, 2022

Conversation

mojtaba-komeili
Copy link
Contributor

Patch description

The Dockerfile for building a working image for ParlAI.

Note

A tutorial on using this Docker image will follow.

Copy link
Contributor

@klshuster klshuster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome, just small comment about torch versioning

Dockerfile Outdated
ENV PATH="/root/anaconda3/bin:$PATH"

# Installing recommmended pre-requirements
RUN conda install pytorch torchvision torchaudio -c pytorch-lts -c nvidia
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we specify torch version? or just assume we work with highest torch number?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, good point. Should we set it to the same version as the install requirements?

pip install -r requirements.txt && \
python setup.py develop

CMD ["parlai", "party"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😍

@mojtaba-komeili mojtaba-komeili merged commit 4bacda3 into main Nov 10, 2022
@mojtaba-komeili mojtaba-komeili deleted the parlai-container branch November 10, 2022 21:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants