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

tutorial pages for expression handling #49

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from
Open

Conversation

kerschke
Copy link

Once PR #1126 from mlr is merged into the master, you can integrate these pages to the tutorial.

@schiffner
Copy link
Contributor

Cool, thx.

@pat-s
Copy link
Collaborator

pat-s commented Jun 22, 2018

@jakob-r Can you give me a short heads-up if the expression handling functionality has ever been implemented? I saw that the first PR was reverted and your second was closed.

If it has not been implemented yet we can also close this one here. No need for a two year old PR that explains something that was rejected from the core.

@jakob-r
Copy link
Contributor

jakob-r commented Jun 22, 2018

It is implemented in ParamHelpers but not in mlr and never will be (too complicated to implement correctly given the architecture of mlr at the moment). So some of the functionality can be used but the tutorial in its form here will never be usefull 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants