search

rlworkgroup / garage

A toolkit for reproducible reinforcement learning research.
MIT License
1.86k stars 310 forks source link

Delete paths_to_tensors #2175

Closed krzentner closed 3 years ago

krzentner commented 3 years ago

This function is no longer needed.

The two remaining uses of it, the results where ignored.

As far as I can tell be reading the history, baselines have never worked in CEM or CMA-ES.

codecov[bot] commented 3 years ago

Codecov Report

Merging #2175 (d2973a4) into master (1c62384) will increase coverage by 0.20%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2175      +/-   ##
==========================================
+ Coverage   91.11%   91.32%   +0.20%     
==========================================
  Files         198      198              
  Lines       11032    10964      -68     
  Branches     1417     1392      -25     
==========================================
- Hits        10052    10013      -39     
+ Misses        717      691      -26     
+ Partials      263      260       -3
Impacted Files Coverage Δ
src/garage/np/_functions.py 76.00% <ø> (-10.07%) :arrow_down:
src/garage/tf/_functions.py 68.85% <ø> (+13.72%) :arrow_up:
src/garage/np/algos/cem.py 100.00% <100.00%> (+2.56%) :arrow_up:
src/garage/np/algos/cma_es.py 100.00% <100.00%> (+3.07%) :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 1c62384...d2973a4. Read the comment docs.