Skip to content

Conversation

@BartekCupial
Copy link
Contributor

Description

This pull request introduces the ability to customize the size of rooms within the MiniGrid MultiRoom environment. Previously, the room size was fixed, but with this change, users can now specify the width and height of the rooms when creating an environment. The default room size remains unchanged, ensuring backward compatibility for existing implementations.

Changes:

  • Added parameters width and height to allow customization of room dimensions in the MiniGrid MultiRoom environment

Motivation:

This enhancement is crucial for integrating MiniGrid environments into MiniHack, which currently encounters errors when environments exceed the dimensions of the NetHack grid. By allowing room size adjustments, this PR facilitates seamless integration and expands the usability of MiniGrid within the MiniHack framework.

Link to PR in MiniHack: #106

@BartekCupial BartekCupial force-pushed the configurable_multiroom branch from f1c9b42 to 9177e8b Compare November 30, 2024 11:36
Copy link
Member

@pseudo-rnd-thoughts pseudo-rnd-thoughts left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, looks good, just added argument documentation

@pseudo-rnd-thoughts pseudo-rnd-thoughts merged commit 939ae21 into Farama-Foundation:master Jan 13, 2025
Martin36 pushed a commit to Martin36/Minigrid that referenced this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants