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

Modify mlagents for removing server build #58

Merged
merged 6 commits into from
Nov 22, 2021

Conversation

leonard-q
Copy link
Collaborator

🌟 Hello! Thanks for contributing JORLDY!

Checklist

Please check if you consider the following items.

  • [v] My code follows the style guidelines of this project contributing
  • [v] My code follows the naming convention of documentation
  • [v] I have commented my code, particularly in hard-to-understand areas
  • [v] My changes generate no new warnings or errors

Types of changes

Please describe the types of changes! (ex. Bugfix, New feature, Documentation, ...)
Improvement

Test Configuration

  • OS: Linux Ubuntu
  • Python version: 3.8
  • Additional libraries:

Description

Please describe the details of your contribution
Modify mlagents env for removing server build. Server build is removed and mlagents envs run with Linux build using no graphics option
It can fix the issue #55

@leonard-q leonard-q requested a review from ramanuzan as a code owner November 22, 2021 04:13
jorldy/core/env/ml_agent.py Outdated Show resolved Hide resolved
jorldy/core/env/ml_agent.py Outdated Show resolved Hide resolved
ramanuzan and others added 2 commits November 22, 2021 13:36
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@leonard-q leonard-q merged commit f15ed82 into master Nov 22, 2021
@leonard-q leonard-q deleted the env/mlagents_server_remove branch November 22, 2021 05:36
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.

2 participants