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.

Sign up for GitHub

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

Jump to bottom

mini issue: please add 1 sentence in the learner API #2327

Closed
pat-s opened this issue Jun 22, 2018 · 2 comments · Fixed by #2348
Closed

mini issue: please add 1 sentence in the learner API #2327

pat-s opened this issue Jun 22, 2018 · 2 comments · Fixed by #2348
Labels
prio-high project - tutorial

Comments

@pat-s
Copy link
Member

pat-s commented Jun 22, 2018

From @berndbischl on September 23, 2015 20:12

Dependent parameters with a \code{requires} field must use \code{quote} and not
\code{expression} to define it.

Copied from original issue: mlr-archive/mlr-tutorial#26
@pat-s
Copy link
Member Author

pat-s commented Jun 22, 2018

From @schiffner on September 23, 2015 20:31

Yep, will do.
Added it to my todo-list when I saw Lars' PR.

@pat-s
Copy link
Member Author

pat-s commented Jun 22, 2018

From @berndbischl on August 27, 2016 12:39

@schiffner
this issues seems to be about 1 sentence. can you do this then close?

@pat-s pat-s mentioned this issue Jun 22, 2018
Closed
@pat-s pat-s mentioned this issue Jul 1, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
prio-high project - tutorial
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add quote() requirement for requires() mlr-org/mlr
2 participants
@schiffner @pat-s