Skip to content

recursion checker proof-of-concept #4560

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
Closed
wants to merge 1 commit into from
Closed

recursion checker proof-of-concept #4560

wants to merge 1 commit into from

Conversation

@chriselion
Copy link
Contributor

@chriselion chriselion commented Oct 12, 2020

Proposed change(s)

Make the approach from #4227 more generic so that we can expand it elsewhere.

Useful links (Github issues, JIRA tickets, ML-Agents forum threads etc.)

Types of change(s)

  • Bug fix
  • New feature
  • Code refactor
  • Breaking change
  • Documentation update
  • Other (please describe)

Checklist

  • Added tests that prove my fix is effective or that my feature works
  • Updated the changelog (if applicable)
  • Updated the documentation (if applicable)
  • Updated the migration guide (if applicable)

Other comments

@vincentpierre
Copy link
Contributor

vincentpierre commented Oct 13, 2020

I think that works. 👍

@chriselion
Copy link
Contributor Author

chriselion commented Oct 19, 2020

Closing in favor of #4573

@chriselion chriselion closed this Oct 19, 2020
@chriselion chriselion deleted the recursion-poc branch May 25, 2021 18:13
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

3 participants

@chriselion @vincentpierre