Closed zsunberg closed 5 years ago
Merging #244 into master will decrease coverage by
0.3%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #244 +/- ##
==========================================
- Coverage 71.88% 71.58% -0.31%
==========================================
Files 10 10
Lines 466 468 +2
==========================================
Hits 335 335
- Misses 131 133 +2
Impacted Files | Coverage Δ | |
---|---|---|
src/pomdp.jl | 80% <0%> (-12.31%) |
: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 cc4cbef...6bccc33. Read the comment docs.
We can emphasize that this is only for simple domains!
Thanks @rejuvyesh , ok I think I'll merge this - if anyone really doesn't like it, we can always go back.
I changed the quick start example in the README to use QuickPOMDPs (see below).
Pros
Cons
I think the pros outweigh the cons for this - does anyone else have an opinion?
from README.md: