fix misleading comment #3058
fix misleading comment #3058
Conversation
| TIMESTEPS_THIS_ITER = "timesteps_this_iter" | ||
|
|
||
| # (Optional/Auto-filled) Accumulated time in seconds for this experiment. | ||
| # (Auto-filled) Accumulated number of timesteps for this entire experiment. |
There was a problem hiding this comment.
Thanks for the catch! Can you put the Optional part back in? Users can override this field.
There was a problem hiding this comment.
This seems ok for consistency with the rest of the auto-filled fields.
There was a problem hiding this comment.
Well, I guess we say "optional" in other places as well, but it's not all accurate. Probably better to just stick with auto-filled.
There was a problem hiding this comment.
Oh yes, sure! I thought that was also obsolete.
There was a problem hiding this comment.
Oh my page wasn't refreshed... now seeing @ericl's comment.
|
Test PASSed. |
## What do these changes do? Fix the misleading comments in code for: - `EPISODES_THIS_ITER` - `EPISODES_TOTAL` Had noted it before and planned to fix it along with some other changes but seemed very relevant to stay next to #3058 so sending this now.
4 participants
What do these changes do?
Fixes the comment for
TIMESTEPS_TOTALwhich seems to refer to an integer.Related issue number
None