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

Fix for alleged type issue with seeding. Closes #368 #370

Merged
merged 2 commits into from
Aug 10, 2020

Conversation

MadcowD
Copy link
Collaborator

@MadcowD MadcowD commented Aug 3, 2020

No description provided.

@MadcowD MadcowD mentioned this pull request Aug 3, 2020
@MadcowD MadcowD changed the title Fix for alleged type issue with seeding. Fix for alleged type issue with seeding. Closes #368 Aug 6, 2020
@MadcowD MadcowD merged commit 5d2a86c into master Aug 10, 2020
@MadcowD MadcowD deleted the MadcowD-patch-1-1 branch August 10, 2020 21:15
MadcowD added a commit that referenced this pull request Aug 18, 2020
* create multiple environment test. starts multiple environments in their own thread

* print done

* gets rid of shutil.Error by not copying the cache.properties.lock

* clean up logging and print statements, make synchronous argument available

* Update core.py to save obs correctly. Closes #365 (#371)

* Fix for alleged type issue with seeding. (#370)

* Initial commit for multiagent implementation

Removing unneeded Render command

Moved _find_server back to where it can still influence the xml.

Correctly tick client/server and process observations.

Branch using correct client-tick obs.

Fixed log once functionality.

Fixed the env stepper

Fixed issues syncing two clients and stepping envs.

Remove dead code.

Moved the multiagent test xmls and script to tests dir.  Fixed action handlers.

obs/ac are now dicts to match the interface of magym.

* Merged in Malmo code changes

* Fixed bugs with TimeHelper and xml loading.

* Fixed a couple issues for multi-agent.

* Update VideoHook.java

Co-authored-by: Julius Frost <33183774+juliusfrost@users.noreply.github.com>
Co-authored-by: William Guss <carlostnb@gmail.com>
Co-authored-by: MineRL Labs <minerl@andrew.cmu.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant