这是indexloc提供的服务,不要输入任何密码
Skip to content

Remove kwargs in learn and update #949

@MischaPanch

Description

@MischaPanch

That's a tricky one since learn implementation are currently not cleanly separated (violate Liskov substitution) and rely on kwargs in order to remain functional. E.g. some implementations of learn take the repeat kwarg (the on-policy ones), and some don't.

Generally, the control flow between trainer.update_policy and policy.learn may need to be redesigned or at least improved

Metadata

Metadata

Assignees

No one assigned

    Labels

    breaking changesChanges in public interfaces. Includes small changes or changes in keysmajorLarge changes that cannot or should not be broken down into smaller onesrefactoringNo change to functionality

    Type

    No type

    Projects

    Status

    Done

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions