Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #11 & #12 #13

Merged
merged 2 commits into from
May 20, 2023
Merged

Fix #11 & #12 #13

merged 2 commits into from
May 20, 2023

Conversation

WhiffleFish
Copy link
Member

  • Add new NativeSARSOP.solve_info that returns the policy as well as an info NamedTuple containing the tree, total planning time, and the total number of planning iterations.
  • Remove unnecessary ba_action field from the tree. This is leftover from when raw POMDP inputs were used for planning instead of the SparseTabular form we use now.

@codecov
Copy link

codecov bot commented May 20, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (a008194) 99.27% compared to head (cddc962) 99.27%.

❗ Current head cddc962 differs from pull request most recent head 9011393. Consider uploading reports for the commit 9011393 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   99.27%   99.27%           
=======================================
  Files          12       12           
  Lines         690      691    +1     
=======================================
+ Hits          685      686    +1     
  Misses          5        5           
Impacted Files Coverage Δ
src/tree.jl 100.00% <ø> (ø)
src/solver.jl 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@WhiffleFish WhiffleFish merged commit 1d77363 into main May 20, 2023
@WhiffleFish WhiffleFish deleted the WhiffleFish-patch1 branch May 20, 2023 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant