Skip to content

Comments

Changes to make it work#7

Open
NikeHop wants to merge 1 commit intoivegner:masterfrom
NikeHop:niklas
Open

Changes to make it work#7
NikeHop wants to merge 1 commit intoivegner:masterfrom
NikeHop:niklas

Conversation

@NikeHop
Copy link

@NikeHop NikeHop commented Mar 23, 2021

I made a few changes, not all of them are relevant.

Overview of the changes:

main.py:

  • Added tensorboard logging
  • outsourced some of the code to utils.py to have a cleaner training file
  • Added some CL arguments to be able to change the exact parameters of the environment

cross_circle_base.py:

  • I contacted the authors and they gave me access to the environment they used. The implementation of your environment is more general so I just changed the default settings to the one they used, a link to their environment is at the beginning of the file

The other changes are as I mentioned in the GitHub issue before

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