Skip to content

Conversation

@MarcCote
Copy link
Collaborator

This pull request introduces several updates to the ScienceWorld project, including documentation improvements, new developer instructions, adjustments to simplification presets, and minor codebase enhancements. The most important changes are grouped below by theme.

Documentation Enhancements:

  • Added a new "Simplifications" section in README.md, explaining the available simplifications and the --simplifications-preset argument, including a warning about differences from the paper's description.
  • Introduced a "Developers" section in README.md with detailed instructions for building the ScienceWorld JAR file, including prerequisites and steps for installation and compilation.

Simplification Preset Updates:

  • Updated the help text for the --simplifications-preset argument in examples/human.py and examples/random_agent.py to specify the exact simplifications applied by the easy preset. [1] [2]

Developer Tooling:

  • Added a new dev extras group in setup.py to include development dependencies from requirements-dev.txt.

Codebase Maintenance:

  • Updated the version in simulator/build.sbt from 1.2.2 to 1.2.3 to reflect the new changes.
  • Refactored the SimplificationOpenContainers logic in SimplifierProcessor.scala to use a more generic EnvObject type instead of Portal for handling container objects.

@MarcCote MarcCote requested a review from PeterAJansen July 13, 2025 18:45
@MarcCote
Copy link
Collaborator Author

Fixes #76

Copy link
Collaborator

@PeterAJansen PeterAJansen left a comment

Choose a reason for hiding this comment

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

Looks good!

@MarcCote MarcCote merged commit 5d569f9 into main Jul 13, 2025
33 checks passed
@MarcCote MarcCote deleted the fix_open_containers_simplification branch July 13, 2025 22:50
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.

3 participants