Closed zsunberg closed 4 years ago
Merging #325 into master will increase coverage by
1.72%
. The diff coverage is89.47%
.
@@ Coverage Diff @@
## master #325 +/- ##
==========================================
+ Coverage 81.96% 83.68% +1.72%
==========================================
Files 8 9 +1
Lines 122 141 +19
==========================================
+ Hits 100 118 +18
- Misses 22 23 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/POMDPs.jl | 100.00% <ø> (ø) |
|
src/common_rl.jl | 89.47% <89.47%> (ø) |
|
src/space.jl | 50.00% <0.00%> (+50.00%) |
: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 d34d217...2da94cb. Read the comment docs.
Actually, I'm kind of thinking that this should go in POMDPModelTools
Yeah POMDPModelTools makes sense too. Best to avoid dependencies here.
Ok, moving to POMDPModelTools
Here is the first step of integration with CommonRLInterface.jl.
What does everyone think? Does this belong in the POMDPs package?