Skip to content
This repository has been archived by the owner on Apr 26, 2023. It is now read-only.

added obs_weight (JuliaPOMDP/POMDPs.jl#172) #71

Merged
merged 1 commit into from
May 23, 2018
Merged

added obs_weight (JuliaPOMDP/POMDPs.jl#172) #71

merged 1 commit into from
May 23, 2018

Conversation

zsunberg
Copy link
Member

also fixed misspelling and removed CircularBuffer

@zsunberg zsunberg requested a review from MaximeBouton May 22, 2018 19:32
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.8%) to 77.918% when pulling d472a5b on obs_weight into 38dbb76 on master.

MaximeBouton
MaximeBouton approved these changes May 23, 2018
View reviewed changes
Copy link
Contributor

@MaximeBouton MaximeBouton left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for doing this.
Nice that you removed the CircularBuffer

@MaximeBouton MaximeBouton mentioned this pull request May 23, 2018
Merged
@zsunberg zsunberg merged commit 4dac41f into master May 23, 2018
@zsunberg zsunberg deleted the obs_weight branch May 23, 2018 18:27
@zsunberg zsunberg mentioned this pull request May 23, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@MaximeBouton MaximeBouton MaximeBouton approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zsunberg @coveralls @MaximeBouton