Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove imitation learning from MPInterface #34

Open
ottofabian opened this issue Jul 6, 2022 · 0 comments
Open

Remove imitation learning from MPInterface #34

ottofabian opened this issue Jul 6, 2022 · 0 comments
Labels
divergent opinions enhancement New feature or request

Comments

@ottofabian
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Learning the MP parameters with imitation learning (IL) is a really specific setting and should not be part of a general MP interface., especially not as a mandatory method to implement for each subclass.
Learning should be part of someone's library using this library.
At the very best, this setting should be included as demo or example how to use the library.

Describe the solution you'd like
Remove the learning from the general interface completely. If it is absolutely necessary, I would only add it as an example how to use the MPs in an IL setting

Describe alternatives you've considered
n.a.

Additional context
n.a.

@ottofabian ottofabian added the enhancement New feature or request label Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
divergent opinions enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants