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

Add TEPPO documentation #2161

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

Add TEPPO documentation #2161

wants to merge 9 commits into from

Conversation

nicolengsy
Copy link
Contributor

To close #2016 .

@nicolengsy nicolengsy requested a review from a team as a code owner October 29, 2020 08:29
@nicolengsy nicolengsy requested review from krzentner and removed request for a team October 29, 2020 08:29
@mergify mergify bot requested review from a team, gitanshu and maliesa96 and removed request for a team October 29, 2020 08:30
@codecov
Copy link

codecov bot commented Oct 30, 2020

Codecov Report

Merging #2161 (51543c2) into master (3b3ad04) will decrease coverage by 0.20%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2161      +/-   ##
==========================================
- Coverage   91.51%   91.31%   -0.21%     
==========================================
  Files         197      198       +1     
  Lines       10864    10944      +80     
  Branches     1370     1374       +4     
==========================================
+ Hits         9942     9993      +51     
- Misses        666      691      +25     
- Partials      256      260       +4     
Impacted Files Coverage Δ
src/garage/sampler/_dtypes.py 90.90% <0.00%> (-9.10%) ⬇️
src/garage/np/_functions.py 77.39% <0.00%> (-7.04%) ⬇️
src/garage/_environment.py 95.45% <0.00%> (-1.99%) ⬇️
src/garage/_dtypes.py 96.01% <0.00%> (-1.19%) ⬇️
src/garage/sampler/vec_worker.py 88.98% <0.00%> (-0.74%) ⬇️
src/garage/trainer.py 90.94% <0.00%> (-0.18%) ⬇️
src/garage/torch/_functions.py 94.68% <0.00%> (-0.06%) ⬇️
src/garage/tf/algos/te.py 100.00% <0.00%> (ø)
src/garage/tf/algos/npo.py 96.46% <0.00%> (ø)
src/garage/torch/algos/bc.py 92.10% <0.00%> (ø)
... and 21 more

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 3b3ad04...51543c2. Read the comment docs.

docs/user/algo_teppo.md Outdated Show resolved Hide resolved
@mergify mergify bot requested a review from a team November 4, 2020 23:49
@gitanshu
Copy link
Contributor

gitanshu commented Nov 9, 2020

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Nov 9, 2020

Command refresh: success

Pull request refreshed

Copy link
Contributor

@krzentner krzentner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please apply Ruofu's fix. Besides that, this change looks good.

@yeukfu
Copy link
Contributor

yeukfu commented Nov 11, 2020

@Mergifyio refresh

@mergify
Copy link
Contributor

mergify bot commented Nov 11, 2020

Command refresh: success

Pull request refreshed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs Page "Algorithms and Methods -> Multi-Task RL -> TEPPO"
5 participants