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

MCTS #386

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

MCTS #386

wants to merge 20 commits into from

Conversation

hades-rp2010
Copy link
Member

Progress -

  • Added modular structure for Tree search agents and tree search planners
  • UCT Node
  • OPD
  • OLOP
  • Trailblazers
  • Platypoos
  • Tests for all

@codecov
Copy link

codecov bot commented Nov 14, 2020

Codecov Report

Merging #386 (1b1bc61) into master (25eb018) will decrease coverage by 4.83%.
The diff coverage is 29.15%.

@@            Coverage Diff             @@
##           master     #386      +/-   ##
==========================================
- Coverage   91.28%   86.45%   -4.84%     
==========================================
  Files          90       95       +5     
  Lines        3809     4126     +317     
==========================================
+ Hits         3477     3567      +90     
- Misses        332      559     +227     
Impacted Files Coverage Δ
genrl/agents/modelbased/mcts/base.py 0.00% <0.00%> (ø)
genrl/agents/modelbased/mcts/mcts.py 0.00% <0.00%> (ø)
genrl/agents/modelbased/mcts/uct.py 0.00% <0.00%> (ø)
genrl/agents/modelbased/base.py 71.42% <71.42%> (ø)
genrl/agents/modelbased/cem/cem.py 94.87% <94.87%> (ø)
genrl/agents/__init__.py 100.00% <100.00%> (ø)
genrl/agents/deep/a2c/a2c.py 93.58% <100.00%> (ø)
genrl/agents/bandits/multiarmed/epsgreedy.py 96.29% <0.00%> (-3.71%) ⬇️
... and 3 more

@lgtm-com
Copy link

lgtm-com bot commented Nov 14, 2020

This pull request introduces 13 alerts when merging 1b1bc61 into 25eb018 - view on LGTM.com

new alerts:

  • 4 for Unused import
  • 3 for Signature mismatch in overriding method
  • 3 for Unused local variable
  • 2 for First parameter of a method is not named 'self'
  • 1 for __init__ method calls overridden method

@hades-rp2010 hades-rp2010 added this to In progress in Model Based RL Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Model Based RL
  
In progress
Development

Successfully merging this pull request may close these issues.

None yet

1 participant