JuliaPOMDP / SARSOP.jl

Julia package that wraps the SARSOP software for solving POMDPs/MDPs
Other
14 stars 7 forks source link

Updates to run() syntax #62

Closed bkraske closed 1 year ago

bkraske commented 1 year ago

Update run() syntax to be threadsafe. See here

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 85.71% and project coverage change: +0.46% :tada:

Comparison is base (7582567) 90.51% compared to head (4e1168d) 90.97%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #62 +/- ## ========================================== + Coverage 90.51% 90.97% +0.46% ========================================== Files 4 5 +1 Lines 137 133 -4 ========================================== - Hits 124 121 -3 + Misses 13 12 -1 ``` | [Files Changed](https://app.codecov.io/gh/JuliaPOMDP/SARSOP.jl/pull/62?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaPOMDP) | Coverage Δ | | |---|---|---| | [src/solver.jl](https://app.codecov.io/gh/JuliaPOMDP/SARSOP.jl/pull/62?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaPOMDP#diff-c3JjL3NvbHZlci5qbA==) | `91.04% <75.00%> (+1.18%)` | :arrow_up: | | [src/graph.jl](https://app.codecov.io/gh/JuliaPOMDP/SARSOP.jl/pull/62?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaPOMDP#diff-c3JjL2dyYXBoLmps) | `68.42% <100.00%> (-1.58%)` | :arrow_down: | | [src/simulator.jl](https://app.codecov.io/gh/JuliaPOMDP/SARSOP.jl/pull/62?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaPOMDP#diff-c3JjL3NpbXVsYXRvci5qbA==) | `100.00% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/JuliaPOMDP/SARSOP.jl/pull/62/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaPOMDP)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

zsunberg commented 1 year ago

Looks good to me. @bkraske is it ready to merge?