Added outputs for the verbose
kwarg
#16
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@printf
. The output format mimics the cpp implementationverbose=true
with simple tests to make sure the header is includedverbose=false
LinearAlgebra
andPrintf
in accordance with https://discourse.julialang.org/t/psa-compat-requirements-in-the-general-registry-are-changing/104958runtests.jl
to be aligned with https://github.com/JuliaDiff/BlueStyleI did not include a version bump with this PR, but I would recommend bumping the version if merged.
Closes #1
@WhiffleFish I wasn't 100% sure if I implemented
# Beliefs
correctly. I am usingsum(.!tree.b_pruned)
to determine the number of beliefs. If this is correct, then it appears NativeSARSOP is using more beliefs than SARSOP which is slowing it down a bit when you compare it on larger problems like TagPOMDPProblem.