JuliaReinforcementLearning / GridWorlds.jl

Help! I'm lost in the flatland!
MIT License
46 stars 9 forks source link

revamp SequentialRoomsUndirected & SequentialRoomsDirected #155

Closed Sid-Bhatia-0 closed 3 years ago

Sid-Bhatia-0 commented 3 years ago
  1. Remove old SequentialRoomsUndirected and SequentialRoomsDirected.
  2. Add new SequentialRoomsUndirectedModule.SequentialRoomsUndirected and SequentialRoomsDirectedModule.SequentialRoomsDirected with improved implementation.
  3. Add agent_direction information to RLBase.state for directed environments when state style is InternalState.
  4. Fix and update tests. Execute RLBase.state inside tests.
  5. Update README.

Here is how the new environments look:

SequentialRoomsUndirectedModule.SequentialRoomsUndirected:

sequential_rooms_undirected

SequentialRoomsDirectedModule.SequentialRoomsDirected:

sequential_rooms_directed

codecov-commenter commented 3 years ago

Codecov Report

Merging #155 (7fbad43) into master (f5a27bf) will decrease coverage by 1.40%. The diff coverage is 76.40%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
- Coverage   73.24%   71.84%   -1.41%     
==========================================
  Files          25       26       +1     
  Lines        2396     2472      +76     
==========================================
+ Hits         1755     1776      +21     
- Misses        641      696      +55     
Impacted Files Coverage Δ
src/envs/envs.jl 72.72% <ø> (-21.03%) :arrow_down:
src/envs/sequential_rooms_directed.jl 57.14% <57.14%> (ø)
src/envs/sequential_rooms_undirected.jl 82.14% <82.14%> (ø)
src/rlbase.jl 82.69% <88.88%> (+24.35%) :arrow_up:
src/grid_world_base.jl 25.64% <0.00%> (-1.29%) :arrow_down:

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 f5a27bf...7fbad43. Read the comment docs.