LARG / HFO

Half Field Offense in Robocup 2D Soccer
MIT License
231 stars 93 forks source link

Cmake revisions; uniform nums; testing scripts - hopefully working with travis #29

Closed drallensmith closed 7 years ago

drallensmith commented 7 years ago

Let's go from simplest to most complicated (or at least most uncertain):

-Allen

drallensmith commented 7 years ago

The second test script, which actually starts up the server, appears to not work with travis due to the virtual environment in use (I have read of other instances of this problem, even with localhost). But at least the first test script can be run automatically.

-Allen

mhauskn commented 7 years ago

Thanks for the PR, these changes look good! In the test script you can enable --fullstate argument to HFO to check that unums come through correctly when there is no partial observability.

Next item is to update the manual to reflect the change in the low-level feature space.

drallensmith commented 7 years ago

Good point re --fullstate (although it was still interesting to see exactly how little information the learner will have at times) and the manual - will try to work on that when I have a bit of time (job interview coming up).