Skip to content

Conversation

@burtenshaw
Copy link
Collaborator

This PR integrates with the hub's built in docker registry via a from_hub method, so you can do:

    env = TextArenaEnv.from_hub(
        "burtenshaw/textarena",
        env_vars={
            "TEXTARENA_ENV_ID": "GuessTheNumber-v0",
            "TEXTARENA_NUM_PLAYERS": "1",
        },
        ports={8000: 8000},
    )

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Nov 3, 2025
@jspisak jspisak added the enhancement New feature or request label Nov 4, 2025
@zkwentz
Copy link
Contributor

zkwentz commented Nov 5, 2025

Awesome.

The only wrinkle we'll have with this, is while we are in a quasi state with Hugging Face docker spaces, the environments pulled will be custom built to have a web interface in a docker space. There may be performance reasons someone would not want that, there might also be specifics on an environment that get overridden because we take over the build on openenv push.

@burtenshaw
Copy link
Collaborator Author

The only wrinkle we'll have with this, is while we are in a quasi state with Hugging Face docker spaces, the environments pulled will be custom built to have a web interface in a docker space. There may be performance reasons someone would not want that, there might also be specifics on an environment that get overridden because we take over the build on openenv push.

Great point. Tbh, I think we could improve the interface component with a feature that accepted a flag like WEB_INTERFACE=0 to block. I've also noticed a lot of community chatter about customising interfaces. I will open an issues on this, and try to encourage some chatter.

@burtenshaw burtenshaw merged commit 3fea97e into main Nov 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants