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.

Sign up for GitHub

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

Jump to bottom

Only send rewards if triggered #137

Merged
merged 21 commits into from
Jul 4, 2016
Merged

Only send rewards if triggered #137

merged 21 commits into from
Jul 4, 2016

Conversation

timhutton
Copy link
Contributor

@timhutton timhutton commented Jul 4, 2016 •
edited
Loading

Also implements support for multi-dimensional rewards. This closes #120.

For Python users, the only breaking change is that reward.value is now reward.getValue().

Tim Hutton and others added 21 commits July 1, 2016 15:06
Added dimension attribute to all reward producers.
71582f6
Using Reward structure defined in schemas for storing rewards. Work i…
2372f0c 
…n progress.
@timhutton
New class MultidimensionalReward.java. Work in progress.
2e38e08
@timhutton
Return of reward message as Reward XML now working. Work in progress.
896ef6f
@timhutton
Made MissionEnded final reward optional, to match behavior with other…
b46a439 
… rewards. Finished add code.
@timhutton
Fix: add(other) wasn't working. Separated clear().
a9c4c15
Storing multidimensional rewards in TimestampedRewards, was Timestamp…
4cefed9 
…edFloat.
Tweaked TimestampedReward API.
9e11f55
Added conversion functions for working with TimestampedReward instanc…
3f77127 
…es from different sources.
Handle parse errors in reward message by adding to world_state.error_…
e5594ac 
…messages.
Merge branch 'master' into rewards
e32be88
Minor: updated changelog with rewards changes.
86ade07
Merge branch 'master' into rewards
11e6703 
Conflicts:
	Minecraft/src/main/java/com/microsoft/Malmo/Client/ClientStateMachine.java
	changelog.txt
Merge branch 'master' into rewards
99b5a28
Merge branch 'master' into rewards
f8209cb
Merge branch 'master' into rewards
4ce21b6
Fix: need to iterate with Map, not HashMap - error on some platforms.
a59b5ed
tabular_q_learning can now use the number of rewards, instead of thei…
cebb40f 
…r value.
Fix: restored mutex guard location to as before.
9f845e3
Minor: printing any errors received is good practice.
e273548
Merge branch 'master' into rewards
3f2a483
@timhutton timhutton merged commit 12767b8 into master Jul 4, 2016
@timhutton timhutton deleted the rewards branch July 4, 2016 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Desired reward behavior: differentiate between no reward and zero reward
1 participant
@timhutton