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

add an option to change timeout before unity responds #5694

Closed
dmitryponv opened this issue Jan 28, 2022 · 9 comments
Closed

add an option to change timeout before unity responds #5694

dmitryponv opened this issue Jan 28, 2022 · 9 comments
Assignees
Labels
request Issue contains a feature request.

Comments

@dmitryponv
Copy link

dmitryponv commented Jan 28, 2022

Is your feature request related to a problem? Please describe.

mlagents-learn times out when Unity play is not pressed. Sometimes unity play takes 30 seconds to start, and MLAgents-learn times out by then

Describe the solution you'd like
Please add a --timeout or some similar option when starting mlagents-learn

@dmitryponv dmitryponv added the request Issue contains a feature request. label Jan 28, 2022
@CubeMD
Copy link

CubeMD commented Jan 30, 2022

Great suggestion. This is also useful as while debugging in Unity editor, the trainer might disconnect on pause

@miguelalonsojr miguelalonsojr self-assigned this Feb 9, 2022
@miguelalonsojr
Copy link
Collaborator

Thanks for reaching out! We've logged this request internally as MLA-2401.

@AVOlight
Copy link

please also support turning the timeout off :)

@danielebanovaz
Copy link

I guess there's still no update on this, right?
Such a simple feature would be extremely useful, especially when starting a large batch of standalone builds

@danielebanovaz
Copy link

I see it has been now "closed as complete".
Does it mean that request MLA-2401 has been fully implemented and it's available in the upcoming release of ML-Agents?

@miguelalonsojr
Copy link
Collaborator

No. Thanks for reaching out. This was closed en masse to clean up old issues and requests. Re-opening as this should be an easy addition. No ETA on implementation though. But will keep open.

@miguelalonsojr miguelalonsojr reopened this Oct 9, 2023
@danielebanovaz
Copy link

Thank you very much for clarifying!

@miguelalonsojr
Copy link
Collaborator

The update was easy enough. Here's the PR: #5991

@miguelalonsojr
Copy link
Collaborator

This feature has been implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
request Issue contains a feature request.
Projects
None yet
Development

No branches or pull requests

5 participants