JuliaReinforcementLearning / GridWorlds.jl

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

revamp GridRoomsUndirected and GridRoomsDirected #154

Closed Sid-Bhatia-0 closed 3 years ago

Sid-Bhatia-0 commented 3 years ago
  1. Remove old GridRoomsUndirected and GridRoomsDirected.
  2. Add new GridRoomsUndirectedModule.GridRoomsUndirected and GridRoomsDirectedModule.GridRoomsDirected.
  3. Rename RLBaseGridWorld to RLBaseEnv.
  4. Fix and update README.
  5. Update tests.

Here is how the new environments look:

GridRoomsUndirectedModule.GridRoomsUndirected:

grid_rooms_undirected

GridRoomsDirectedModule.GridRoomsDirected:

grid_rooms_directed

codecov-commenter commented 3 years ago

Codecov Report

Merging #154 (f4bee64) into master (c585c20) will decrease coverage by 1.11%. The diff coverage is 73.03%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #154      +/-   ##
==========================================
- Coverage   74.35%   73.24%   -1.12%     
==========================================
  Files          24       25       +1     
  Lines        2379     2396      +17     
==========================================
- Hits         1769     1755      -14     
- Misses        610      641      +31     
Impacted Files Coverage Δ
src/play.jl 0.00% <0.00%> (ø)
src/rlbase.jl 58.33% <57.14%> (-6.38%) :arrow_down:
src/envs/grid_rooms_directed.jl 71.42% <71.42%> (ø)
src/envs/envs.jl 93.75% <77.77%> (-6.25%) :arrow_down:
src/envs/grid_rooms_undirected.jl 81.81% <81.81%> (ø)
src/envs/dynamic_obstacles.jl 92.40% <0.00%> (+0.04%) :arrow_up:

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 c585c20...f4bee64. Read the comment docs.