You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Okay, I see why you need these "hacks". Ideally what we want here is:
General support for LR scheduling. Therefore the lr scheduler params should move out of the policy config.
Policies have an optional get_optimizer_param_dicts. Might need more thinking through for the general design.
Maybe we can keep these ideas out of this PR though. But what are your thoughts?
get_optimizer_param_dicts
. Might need more thinking through for the general design.Maybe we can keep these ideas out of this PR though. But what are your thoughts?
Originally posted by @alexander-soare in #99 (comment)
The text was updated successfully, but these errors were encountered: