huggingface / huggingface/openenv-course

Bug: Broken pip install links in Module 1 Notebook (Exit code 128 & Missing pyproject.toml)

Open
#4 0 comments 1 reaction 0 assignees View on GitHub
Dominant language
Jupyter Notebook
Stars
88
Forks
157
PR merge metrics
No merged PRs in 30d

Description

Hey team,

Was grinding through Module 1 for the hackathon today and bumped into a few broken ""!pip install" links in the Colab notebook.

What is happening:
1. The Hugging Face Spaces links for 'Echo' and 'Catch' are throwing an `exit code: 128`
2. The TextArena Wordle link fails because `pip` can't find a `setup.py` or `pyproject.toml` file in that specific repo.

The Fix:
I managed to get everything running smoothly by bypassing the HF Spaces entirely and pulling the client code directly from the official Meta GitHub using the `subdirectory` flag.

Here are the working replacements if you want to update the notebook for the rest of the cohort:

Change these:
`!pip install -q git+https://huggingface.co/spaces/openenv/echo-env`
`!pip install -q git+https://huggingface.co/spaces/openenv/openspiel-env`
`!pip install -q git+https://huggingface.co/spaces/burtenshaw/textarena`

To these:
`!pip install -q "git+https://github.com/meta-pytorch/OpenEnv.git#subdirectory=envs/echo_env"`
`!pip install -q "git+https://github.com/meta-pytorch/OpenEnv.git#subdirectory=envs/openspiel_env"`
`!pip install -q "git+https://github.com/meta-pytorch/OpenEnv.git#subdirectory=envs/textarena_env"`

*(Just a heads-up: using this fix means the Python imports in the next cells need the `envs.` prefix dropped, e.g., changing `from envs.openspiel_env import OpenSpielEnv` to `from openspiel_env import OpenSpielEnv`).*

Hope this helps save some time for everyone else jumping into the hackathon. Let me know if you want me to just open a PR for this!

Cheers.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.