Closed nathan-miller23 closed 3 years ago
OvercookedEnv.featurize_state_mdp
were passing in self.horizon
as the num_pots
param, resulting in massive featurized vector sizesovercooked_ai_py/mdp/overcooked_trajectory.py
file
featurize_state
that caused pots to be incorrectly encoded when num_pots != 2
featurize_state
that iterate over num_pots
mdp.featurize_state_shape
(and mdp.lossless_state_encoding
) property
mdp.get_featurize_state_shape
(and mdp.get_lossless_state_encoding_shape
)num_pots
can be passed into the featurize_shape method, as num_pots
impacts encoding shapeMerging #76 (26f82c6) into master (242c24d) will decrease coverage by
0.68%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #76 +/- ##
==========================================
- Coverage 82.70% 82.01% -0.69%
==========================================
Files 10 16 +6
Lines 3035 3636 +601
==========================================
+ Hits 2510 2982 +472
- Misses 525 654 +129
Flag | Coverage Δ | |
---|---|---|
no-planners | ? |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
overcooked_ai_py/agents/benchmarking.py | 56.90% <0.00%> (-9.39%) |
:arrow_down: |
overcooked_ai_py/planning/planners.py | 84.99% <0.00%> (-0.85%) |
:arrow_down: |
overcooked_ai_py/mdp/overcooked_mdp.py | 92.00% <0.00%> (-0.81%) |
:arrow_down: |
overcooked_ai_py/static.py | 100.00% <0.00%> (ø) |
|
...rcooked_ai_py/visualization/visualization_utils.py | 41.66% <0.00%> (ø) |
|
overcooked_ai_py/visualization/pygame_utils.py | 69.01% <0.00%> (ø) |
|
overcooked_ai_py/mdp/overcooked_trajectory.py | 76.00% <0.00%> (ø) |
|
overcooked_ai_py/__init__.py | 100.00% <0.00%> (ø) |
|
overcooked_ai_py/utils.py | 58.97% <0.00%> (ø) |
|
overcooked_ai_py/visualization/state_visualizer.py | 95.97% <0.00%> (ø) |
|
... and 3 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3b7a161...26f82c6. Read the comment docs.
Updates in
featurize_state
to account for both tomatoes and variable length potsfeaturize_state
now takes anum_pots
parameter. The state of the closestnum_pots
pots is encoded into higher dimensional featuresnum_pots
pots intead