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

MetaEvaluate with multiple trajectories #1152

Merged
merged 1 commit into from
Jan 25, 2020
Merged

Conversation

krzentner
Copy link
Contributor

This is necesary because some methods use multiple exploration
trajectories for each task.

This is necesary because some methods use multiple exploration
trajectories for each task.
@krzentner krzentner requested a review from a team as a code owner January 24, 2020 22:59
@codecov
Copy link

codecov bot commented Jan 24, 2020

Codecov Report

Merging #1152 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1152      +/-   ##
==========================================
+ Coverage   87.26%   87.27%   +0.01%     
==========================================
  Files         180      180              
  Lines        8575     8576       +1     
  Branches     1112     1113       +1     
==========================================
+ Hits         7483     7485       +2     
  Misses        896      896              
+ Partials      196      195       -1
Impacted Files Coverage Δ
src/garage/experiment/meta_evaluator.py 90.9% <100%> (+0.43%) ⬆️
src/garage/sampler/worker.py 94.23% <0%> (+0.96%) ⬆️

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 555dcc7...3e0d69c. Read the comment docs.

@mergify mergify bot merged commit e14e811 into master Jan 25, 2020
@mergify mergify bot deleted the meta_evaluator_n_traj branch January 25, 2020 01:28
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.

None yet

3 participants