search

JuliaReinforcementLearning / GridWorlds.jl

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

revamp TransportUndirected & TransportDirected #168

Closed Sid-Bhatia-0 closed 3 years ago

Sid-Bhatia-0 commented 3 years ago
  1. Remove old TransportUndirected and TransportDirected.
  2. Add new TransportUndirectedModule.TransportUndirected and TransportDirectedModule.TransportDirected.
  3. Update tests.
  4. Update README.

Here is how the new environments look:

TransportUndirectedModule.TransportUndirected:

transport_undirected

TransportDirectedModule.TransportDirected:

transport_directed

codecov-commenter commented 3 years ago

Codecov Report

Merging #168 (992c9c0) into master (31cce74) will decrease coverage by 1.08%. The diff coverage is 79.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   66.97%   65.89%   -1.09%     
==========================================
  Files          32       33       +1     
  Lines        2577     2577              
==========================================
- Hits         1726     1698      -28     
- Misses        851      879      +28
Impacted Files Coverage Δ
src/envs/envs.jl 72.72% <ø> (ø)
src/envs/transport_directed.jl 72.41% <72.41%> (ø)
src/envs/transport_undirected.jl 82.71% <82.71%> (ø)
src/rlbase.jl 86.11% <87.50%> (+0.13%) :arrow_up:
src/actions.jl 59.09% <0.00%> (-20.46%) :arrow_down:
src/envs/catcher.jl 76.31% <0.00%> (-3.95%) :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 31cce74...992c9c0. Read the comment docs.