Skip to content
This repository was archived by the owner on May 6, 2021. It is now read-only.

Conversation

@findmyway
Copy link
Member

Now trajectory is fully composable. This will make my life much easier to implement JuliaReinforcementLearning/ReinforcementLearningZoo.jl#60

@codecov-commenter
Copy link

Codecov Report

Merging #106 into master will increase coverage by 2.66%.
The diff coverage is 83.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
+ Coverage   42.09%   44.76%   +2.66%     
==========================================
  Files          41       33       -8     
  Lines         803      735      -68     
==========================================
- Hits          338      329       -9     
+ Misses        465      406      -59     
Impacted Files Coverage Δ
src/components/agents/dyna_agent.jl 0.00% <ø> (ø)
src/utils/circular_array_buffer.jl 87.80% <ø> (-2.44%) ⬇️
src/components/trajectories/trajectory.jl 80.85% <80.43%> (-2.49%) ⬇️
src/components/agents/agent.jl 72.09% <88.88%> (+29.47%) ⬆️
src/components/trajectories/abstract_trajectory.jl 100.00% <100.00%> (+35.29%) ⬆️
src/utils/sum_tree.jl 49.15% <100.00%> (+8.47%) ⬆️
src/extensions/Flux.jl 7.69% <0.00%> (+7.69%) ⬆️

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 ed70834...5713c1f. Read the comment docs.

@findmyway findmyway merged commit b855f9f into JuliaReinforcementLearning:master Aug 12, 2020
@findmyway findmyway deleted the trajectory_redesign branch August 12, 2020 17:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants