Closed AdamGleave closed 4 years ago
Merging #47 into master will increase coverage by
0.04%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #47 +/- ##
==========================================
+ Coverage 85.10% 85.14% +0.04%
==========================================
Files 65 65
Lines 4263 4262 -1
==========================================
+ Hits 3628 3629 +1
+ Misses 635 633 -2
Impacted Files | Coverage Δ | |
---|---|---|
src/evaluating_rewards/envs/point_mass.py | 83.33% <ø> (ø) |
|
src/evaluating_rewards/policies/monte_carlo.py | 0.00% <ø> (ø) |
|
src/evaluating_rewards/__init__.py | 100.00% <100.00%> (ø) |
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 49639e3...7ee37c0. Read the comment docs.
Delete
runners/comparison/hardcoded_npec.sh
andrunners/comparison/learnt.sh
which are now redundant with the newscripts.distances.npec
.Delete
runners/comparison/hardcoded_epic_sweep.sh
andrunners/comparison/hardcoded_figs.sh
that are non-functional after refactoring theplot_{epic,npec,erc}_heatmap
code. These may still be useful as a starting point for similar experiments in the future.Delete
runners/eval/greedy_pm_irl.sh
andrunners/eval/learnt.sh
that are also non-functional but may be useful as a starting point in the future.Fix
runners/eval/greedy_pm_hardcoded.sh
and it's Monte Carlo dependents. This is now tested working.Note
irl/train_irl.sh
andtransfer_point_maze.sh
are unlikely to work verbatim, but I intend to address those in separate PRs.