fix(curriculum): Make boundaries in curriculum more sensible (#407)

* init

* fix tests

* unify codeio

* filtered for libraries not present in reasoning-gym

* fix more bounds

* puzzle24

* knight swap curriculum

* fix number sorting

* fix attributes

* add validation of config in creation of dataset

* dry run for instantiating and validating the datasets

* remove unused imports

* fix curriculum tests to reference newly updated attribute names
This commit is contained in:
Zafir Stojanovski 2025-04-04 20:24:14 +02:00 committed by GitHub
parent 7853263650
commit dced3bfc45
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
132 changed files with 1226 additions and 347 deletions

View file

@ -126,18 +126,18 @@ def test_ransom_note_curriculum():
base_cfg: RansomNoteConfig = curriculum.generate_configuration(base_value)
assert base_cfg.seed == 1
assert base_cfg.size == 150
assert base_cfg.min_note_length == 10 and base_cfg.max_note_length == 10
assert base_cfg.min_magazine_length == 50 and base_cfg.max_magazine_length == 50
assert base_cfg.min_note_length == 10 and base_cfg.max_note_length == 50
assert base_cfg.min_magazine_length == 50 and base_cfg.max_magazine_length == 100
# test incrementing attribute levels
curriculum.increment_attr_level("note_length")
curriculum.increment_attr_level("magazine_length")
increased_cfg = curriculum.generate_configuration(base_value)
assert increased_cfg.min_note_length == 10 and increased_cfg.max_note_length == 50
assert increased_cfg.min_magazine_length == 50 and increased_cfg.max_magazine_length == 100
assert increased_cfg.min_note_length == 10 and increased_cfg.max_note_length == 100
assert increased_cfg.min_magazine_length == 50 and increased_cfg.max_magazine_length == 500
# test decrementing attribute level for note_length again
curriculum.decrement_attr_level("note_length")
partially_decreased_cfg = curriculum.generate_configuration(base_value)
assert partially_decreased_cfg.min_note_length == 10 and partially_decreased_cfg.max_note_length == 10
assert partially_decreased_cfg.min_magazine_length == 50 and partially_decreased_cfg.max_magazine_length == 100
assert partially_decreased_cfg.min_note_length == 10 and partially_decreased_cfg.max_note_length == 50
assert partially_decreased_cfg.min_magazine_length == 50 and partially_decreased_cfg.max_magazine_length == 500