Try to fix normal runtime error in the training phase#66
Open
MiaoDX wants to merge 1 commit intounitreerobotics:mainfrom
Open
Try to fix normal runtime error in the training phase#66MiaoDX wants to merge 1 commit intounitreerobotics:mainfrom
MiaoDX wants to merge 1 commit intounitreerobotics:mainfrom
Conversation
Signed-off-by: MiaoDX <miaodx@hotmail.com>
Collaborator
|
Ok, I’ll update a more stable version later. But for the current flat-ground env, the training is already completed before the program exits, so you can just pick a converged iteration directly. |
|
Hi, changing the limit_angle doesn't fix the issue in my case. It seams to occur randomly after 50k iterations and I can't figure out why. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi,
As discussed in #24
It seems G1 velocity demo is not working properly, please have a check.
As zhenwang-civil suggested, change
bad_orientation = DoneTerm(func=mdp.bad_orientation, params={"limit_angle": 0.8})from 0.8 to 0.7 does fix this problem.But I not so sure if this the best solution, after dig into the doc and help from codex/claude, it seems we can also change the loss function def.
But still, I am new to this area, please give some commits @Agnel-Wang
Sincerely,
MiaoDX