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

Update ReadMe to recommend poetry install for Mephisto #4612

Merged
merged 2 commits into from
Jul 5, 2022

Conversation

dexterju27
Copy link
Contributor

Patch description

  1. Mephisto has known compatibly issue with ParlAI due to their pinned dependencies. An easy walkaround would be install
    Mephisto via poetry.
    Testing steps

Other information

@@ -1,6 +1,6 @@
# Crowdsourcing tasks

Code for crowdsourcing tasks that use Mephisto. See the [Mephisto quick start guide](https://github.com/facebookresearch/mephisto/blob/main/docs/quickstart.md) to quickly get started with Mephisto. This README provides a quick overview of how to run crowdsourcing tasks in ParlAI: see the [tutorial](https://github.com/facebookresearch/ParlAI/blob/main/docs/source/tutorial_crowdsourcing.md) for a deeper guide on how to set up and configure new crowdsourcing tasks.
Code for crowdsourcing tasks that use Mephisto. See the [Mephisto quick start guide](https://mephisto.ai/docs/guides/quickstart/) to quickly get started with Mephisto. **Please install Mephisto via poetry to avoid depdency conflits with ParlAI.** This README provides a quick overview of how to run crowdsourcing tasks in ParlAI: see the [tutorial](https://github.com/facebookresearch/ParlAI/blob/main/docs/source/tutorial_crowdsourcing.md) for a deeper guide on how to set up and configure new crowdsourcing tasks.
Copy link
Contributor

Choose a reason for hiding this comment

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

# install poetry
$ curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python
# from the root dir, install Mephisto:
$ poetry install

@dexterju27
Copy link
Contributor Author

Ping : )

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.

sorry thought I approved

@dexterju27
Copy link
Contributor Author

Thanks!

@dexterju27 dexterju27 merged commit 467ad37 into main Jul 5, 2022
@dexterju27 dexterju27 deleted the dexterju27-patch-1 branch July 5, 2022 17:59
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